-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add Enhanced MLOS Analyzer Dashboard with Advanced Visualizations as a Library #944
base: main
Are you sure you want to change the base?
Conversation
…alysis A library for analysis
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
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 think we discussed putting these into the mlos_viz
package.
I don't think we need to introduce a new package for these.
My understanding is it was going to be part of "mlos_webui" or something like this and it should be seperate based on #838 I also assumed it would someway be put into mlos_viz later on I can reformat later this month |
To my mind most of what's here are the data analysis/visualization routines. Those should go in mlos_viz. After that, folks can choose to call them in either a notebook or we can make use of them in the mlos_webui, but that will be a separate PR. Make sense? Also, remember that we'll want to enable all the usual code quality checks throughout these (e.g., unit tests, type checks, linting, formatting, etc.). Thanks! |
Add Enhanced MLOS Analyzer Dashboard with Advanced Visualizations
Added comprehensive dashboard for MLOS experiment analysis with new visualization modules:
Type of Change:
Testing:
Additional Notes:
Code organization follows src layout pattern. Added type hints and docstrings for maintainability.
Example Usage