-
Notifications
You must be signed in to change notification settings - Fork 51
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
Change behavior of the problem_recorder and driver_recorder options on dashboard command #320
Change behavior of the problem_recorder and driver_recorder options on dashboard command #320
Conversation
aviary/visualization/dashboard.py
Outdated
driver_recorder : str | ||
Name of the recorder file containing the Driver cases. | ||
driver_recorder : str or None | ||
Name of the recorder file containing the Driver cases. If None, ignore this file |
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'd prefer something more descriptive than "ignored", maybe something like:
Name of the recorder file containing the Driver cases. If None, ignore this file | |
Name of the recorder file containing the Driver cases. If None, the driver tab will not be added |
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.
Good suggestion! Fixed
#330 changed aviary/docs/user_guide/outputs_and_how_to_read_them from .md to .ipynb, if you need help resolving the merge conflict, let me know. |
…hange-recorder-name-dashboard-option-behavior
@hschilling Just created a PR to your branch to resolve the merge conflict, which should fix the doc failure |
Thanks! |
You'll have to approve it here |
Changed default name of problem recorder file to problem_history.db
Summary
Modified the way the dashboard command works so that the dashboard always uses the problem and driver case recorder files and that the problem_recorder and driver_recorder options be used to indicate that a non-standard name for the file was used. The exception to that is if driver_recorder is set to None, which would indicate not reading the driver recorder file, which could be very large and the resulting dashboard tab not of interest to the user.
Also changed the name of the default filenames to be
problem_history.db
anddriver_history.db
.Related Issues
Backwards incompatibilities
None
New Dependencies
None