Skip to content

Implement multiple continous choices and Druedahl Jorgensen upper envelope#198

Open
MaxBlesch wants to merge 79 commits into
mainfrom
multi_cont
Open

Implement multiple continous choices and Druedahl Jorgensen upper envelope#198
MaxBlesch wants to merge 79 commits into
mainfrom
multi_cont

Conversation

@MaxBlesch
Copy link
Copy Markdown
Member

No description provided.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@amageh amageh added the enhancement New feature or request label Apr 22, 2026
@MaxBlesch MaxBlesch requested a review from segsell April 24, 2026 11:00
Comment thread src/dcegm/pre_processing/setup_model.py Outdated
Comment thread src/dcegm/pre_processing/model_structure/model_structure.py Outdated
Comment thread src/dcegm/pre_processing/model_structure/model_structure.py Outdated
Comment thread src/dcegm/pre_processing/model_structure/model_structure.py Outdated
Comment thread docs/source/background/two_occupations.ipynb
Comment thread tests/test_two_occupation_model.py Outdated
@MaxBlesch
Copy link
Copy Markdown
Member Author

I addressed the comments by @segsell. When doing so, I reran the notebook by @amageh (there are two right now about two occupations, we still need to delete the wrong one). I realized that the results for me when running it did not look as before. Even without changing anything. Could you maybe rerun @amageh and see if it is my machine maybe?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants