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

Make minor changes and add docs for structural reforms #5427

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

anth-volk
Copy link
Contributor

This is a draft PR meant only for local testing purposes and should not be merged

@PavelMakarchuk I've worked up a Jupyter notebook in docs/usage/new_structural_reforms.ipynb explaining this first draft version of a new structural reform interface. I'd love your feedback and hope this doc serves as a guide as we continue refining this implementation.

@PavelMakarchuk
Copy link
Collaborator

Could you create a Colab Notebook with an example reform which can be accessed when checking out this PR - this can give me some room to test out different reform structures and edge cases but based on our conversation - the implementation will definitely simplify the reform creating process

@anth-volk
Copy link
Contributor Author

While I could create a Colab notebook, I'm not sure if it'd be possible to actually run the code, since successfully running this requires installing two locally modified packages, -us and -core.

@anth-volk
Copy link
Contributor Author

Forgot to write the second half of this. Would that be useful to you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants