-
Notifications
You must be signed in to change notification settings - Fork 30
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
Which Optimizer to Choose #536
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Codecov ReportAll modified and coverable lines are covered by tests ✅ |
for more information, see https://pre-commit.ci
…Economics/optimagic into how_to_algo_selection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very neat! Just textual changes / a bit of structure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it a lot!
Some small comments and an idea how to simplify the structure below.
Co-authored-by: Hans-Martin von Gaudecker <[email protected]>
Co-authored-by: Christian Zimpelmann <[email protected]>
Thanks @ChristianZimpelmann and @hmgaudecker for the feedback. I think this made the guide much clearer! |
Update the
how_to_algorithm_selection
notebook to include guidelines for algorithm selection based on theory.