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

DABL notebook integration #41

Closed
wants to merge 20 commits into from
Closed

Conversation

bpkroth
Copy link
Collaborator

@bpkroth bpkroth commented Jan 3, 2024

Makes use of microsoft/MLOS#624 to plot results in the demo notebooks via the simplified mlos_viz.plot() API.

Keeps the old visualization demonstrations as well, but moves them towards the bottom of the notebook to prioritize the new API's simplicity.

  • Update TODO text in teacher notebook with takeaways

@bpkroth bpkroth requested a review from motus January 12, 2024 23:48
@bpkroth bpkroth added the WIP Work in progress label Jan 12, 2024
@bpkroth
Copy link
Collaborator Author

bpkroth commented Jan 13, 2024

@motus this is just waiting on microsoft/MLOS#624 now.

bpkroth added a commit to microsoft/MLOS that referenced this pull request Jan 16, 2024
Adds a basic `mlos_viz.plot(exp)` style API for simple visualizations of
`ExperimentData` results relative to the experiment's objectives
(building off of #628 and dabl/dabl#335).

Note: this PR currently omits unit tests for the new module due to the
complexity of testing visualizations. We intend to add this in future
PRs. There is however, a working example of its use here right now:
Microsoft-CISL/sqlite-autotuning#41

---------

Co-authored-by: Sergiy Matusevych <sergiy.matusevych@gmail.com>
@bpkroth
Copy link
Collaborator Author

bpkroth commented Feb 1, 2024

Moving to #43

@bpkroth bpkroth closed this Feb 1, 2024
@bpkroth bpkroth mentioned this pull request Feb 1, 2024
Merged
bpkroth added a commit that referenced this pull request Feb 1, 2024
Includes updates from #41 for dabl plotting integration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant