-
Notifications
You must be signed in to change notification settings - Fork 65
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
bokeh timeseries plots now comply with OpenMDAO's placement of recorder files. #1106
Conversation
…record files in the problem outputs directory. Handle older OpenMDAO versions.
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Added clean_notebooks script for quicly cleaning outputs from all notebooks.
…ion() in tests to handle older versions.
@@ -31,7 +31,7 @@ jobs: | |||
SCIPY: '1.13' | |||
PETSc: '3.19' | |||
PYOPTSPARSE: 'v2.11.0' | |||
OPENMDAO: 'latest' | |||
OPENMDAO: 'dev' |
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.
Changed to dev to avoid backwards compatibility code in the documentation.
description: Verifies that documentation notebooks contain no output cells. | ||
language: python | ||
pass_filenames: false | ||
entry: python ./docs/clean_notebooks.py --dryrun ./docs |
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.
Added to prevent accidentally commit of unclean notebooks.
@@ -253,3 +255,26 @@ def create_subprob(base_name, comm, reports=False): | |||
str_hash = hashlib.sha256(used_for_security=False)[:8] | |||
p = om.Problem(comm=comm, reports=reports, name=f'{sim_prob_name}_{str_hash}') | |||
return p | |||
|
|||
|
|||
def om_version(): |
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.
Removes a lot of duplicate code of converting OpenMDAO version to tuple for comparison.
[tool.ruff.lint.pycodestyle] | ||
max-line-length = 100 | ||
max-line-length = 130 |
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.
Consistent with pycodestyle settings.
Still need some work to implement pycodestyle checks.
Summary
The latest OpenMDAO change puts recorder files in the problem outputs directory by default. This change makes dymos' timeseries plots handle that correctly.
Related Issues
Backwards incompatibilities
None
New Dependencies
None