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

Timeseries report tooltip hover change. #1098

Merged
merged 2 commits into from
Aug 13, 2024

Conversation

robfalck
Copy link
Contributor

Summary

When using the bokeh timeseries report, the tooltip hover will now show the value of the nearest data point, rather than the value under the cursor.

Addressed timeeseries report test failure that fails if environment variable OPENMDAO_REPORTS=0. This test now temporarily sets the value to 1 and runs. Note that this function isn't a true report, although it hijacks the report machinery to place the resulting file in the reports directory. This function can be run offline from record files, so when reports are not active it assumes that it is running in this mode and outputs the file to the current working directory.

Related Issues

Backwards incompatibilities

None

New Dependencies

None

… values, no value under cursor.

Addressed timeeseries report test failure that fails if environment variable OPENMDAO_REPORTS=0.
This test now temporarily sets the value to 1 and runs.
Note that this function isn't a true report, although it hijacks the report machinery to place the resulting file in the reports directory.
This function can be run offline from record files, so when reports are not active it assumes that it is running in this mode and outputs the file to the current working directory.
@coveralls
Copy link

Coverage Status

coverage: 92.525% (-0.2%) from 92.73%
when pulling 1684b8f on robfalck:bokeh_hover_update
into f56e0e9 on OpenMDAO:master.

@robfalck robfalck merged commit 220c89e into OpenMDAO:master Aug 13, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants