Skip to content

Add AutoML Interactive Extension - #6243

Merged
JakeRadMSFT merged 12 commits into
mainfrom
u/jakerad/addAutoMLInteractive
Jul 6, 2022
Merged

Add AutoML Interactive Extension#6243
JakeRadMSFT merged 12 commits into
mainfrom
u/jakerad/addAutoMLInteractive

Conversation

@JakeRadMSFT

@JakeRadMSFTJakeRadMSFT commented Jun 30, 2022

Copy link
Copy Markdown
Member

This PR is going to add a Visualizer for AutoML experiments to track AutoML progress while it's running in a Notebook.

It's not perfect but it's a start:

image


So we can do the best job, please check:

  • There's a descriptive title that will make sense to other developers some time from now.
  • There's associated issues. All PR's should have issue(s) associated - unless a trivial self-evident change such as fixing a typo. You can use the format Fixes #nnnn in your description to cause GitHub to automatically close the issue(s) when your PR is merged.
  • Your change description explains what the change does, why you chose your approach, and anything else that reviewers should know.
  • You have included any necessary tests in the same PR.

@JakeRadMSFT
Initial check-in of AutoML Interactive Extension
@JakeRadMSFT
Remove waring disable code
@JakeRadMSFT
clean up warnings
@JakeRadMSFT

JakeRadMSFT commented Jun 30, 2022

Copy link
Copy Markdown
MemberAuthor

@codecov

codecovBot commented Jun 30, 2022

Copy link
Copy Markdown

Codecov Report

Merging #6243 (02ffadd) into main (ea2dc6e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@ Coverage Diff @@## main #6243 +/- ##
=======================================
Coverage 68.39% 68.39% =======================================
Files 1141 1141 Lines 244790 244790 Branches 25405 25405 =======================================
+ Hits 167419 167430 +11 + Misses 70715 70705 -10 + Partials 6656 6655 -1 
FlagCoverage Δ
Debug68.39% <ø> (+<0.01%)⬆️
production62.85% <ø> (+<0.01%)⬆️
test88.99% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted FilesCoverage Δ
src/Microsoft.ML.Data/Transforms/ColumnCopying.cs96.26% <0.00%> (-0.94%)⬇️
...StandardTrainers/Standard/LinearModelParameters.cs65.82% <0.00%> (-0.26%)⬇️
...soft.ML.Transforms/Text/WordEmbeddingsExtractor.cs86.87% <0.00%> (+1.13%)⬆️
...c/Microsoft.ML.FastTree/Utils/ThreadTaskManager.cs100.00% <0.00%> (+20.51%)⬆️

@JakeRadMSFT
Add chart header and work around Plotly.NET issue.
@JakeRadMSFT
Fix chart title.
@JakeRadMSFTJakeRadMSFT changed the title WIP - Add AutoML Interactive ExtensionAdd AutoML Interactive ExtensionJul 1, 2022
@JakeRadMSFT
Improve styling and add header to Table.
Comment threadsrc/Microsoft.ML.AutoML.Interactive/NotebookMonitor.cs
Comment threadsrc/Microsoft.ML.AutoML/Microsoft.ML.AutoML.csproj Outdated
Comment threadsrc/Microsoft.ML.AutoML/Microsoft.ML.AutoML.csproj
@JakeRadMSFT
Update comments and remove some items from NoWarn.
Comment threadsrc/Microsoft.ML.AutoML.Interactive/NotebookMonitor.cs Outdated
@JakeRadMSFT
Remove trailing semicolon
Comment threadsrc/Microsoft.ML.AutoML.Interactive/NotebookMonitor.cs Outdated
@JakeRadMSFT
Resolve PR feedback
@JakeRadMSFT
Move _updatePending = false outside of if check.

@LittleLittleCloudLittleLittleCloud left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment threadsrc/Microsoft.ML.AutoML.Interactive/NotebookMonitor.cs Outdated
@JakeRadMSFT
Resolve PR Feedback
Comment threadsrc/Microsoft.ML.AutoML.Interactive/NotebookMonitor.cs Outdated
@JakeRadMSFT
Pull out ActionThrottler into class and fix delay bug.
@JakeRadMSFT
JakeRadMSFT merged commit 924ae7a into mainJul 6, 2022
@ghostghost locked as resolved and limited conversation to collaborators Aug 5, 2022
@michaelgsharp
michaelgsharp deleted the u/jakerad/addAutoMLInteractive branch May 10, 2023 20:29
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@JakeRadMSFT@tarekgh@LittleLittleCloud@michaelgsharp