Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add process to automatically update example notebooks #1328

Merged
merged 10 commits into from
Jul 31, 2023

Conversation

AA-Turner
Copy link
Collaborator

cc: @MridulS

This is the promised alternative to #1322.

This workflow strips all output from example notebooks and then re-executes them all, before opening a PR with the changes. I chose to have the workflow open a PR rather than committing directly itself to allow reviewer/human intervention if need be.

The workflow will only run on changes to the examples directory -- I thought this was a better tradeoff than having it run on every change to the core HARK code, though of course changes to core HARK code will change outputs in the example notebooks.

A

@MridulS
Copy link
Member

MridulS commented Jul 31, 2023

This workflow strips all output from example notebooks and then re-executes them all, before opening a PR with the changes. I chose to have the workflow open a PR rather than committing directly itself to allow reviewer/human intervention if need be.

I like this, thanks!

Let's try running this :)

@MridulS MridulS merged commit a547d08 into econ-ark:master Jul 31, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants