Skip to content

Conversation

@daavoo
Copy link
Contributor

@daavoo daavoo commented Jan 26, 2023

@daavoo daavoo marked this pull request as draft January 26, 2023 17:23
@daavoo daavoo requested a review from alex000kim January 26, 2023 17:24
@daavoo daavoo requested a review from shcheklein January 26, 2023 17:31
shcheklein

This comment was marked as resolved.

@daavoo

This comment was marked as resolved.

@dberenbaum

This comment was marked as resolved.

@daavoo daavoo force-pushed the example-get-started-cv branch from 09393dc to dbea15a Compare January 31, 2023 15:50
@daavoo daavoo self-assigned this Jan 31, 2023
@daavoo daavoo marked this pull request as ready for review January 31, 2023 19:02

pip install -r requirements.txt
pip install jupyter
jupyter nbconvert --execute 'notebooks/TrainSegModel.ipynb' --inplace
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This needs to be addressed from a product perspective but leaving it here.

Enabling live metrics in this onboarding step it's kind of awkward / not possible because without having the dvc.yaml committed and pushed Studio can't display the metrics, params, and plots.

Copy link
Member

Choose a reason for hiding this comment

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

It makes sense for VS Code, right?

Is there a ticket in Studio for this btw?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It makes sense for VS Code, right?

For VSCode it just works because the files are locally available.

Is there a ticket in Studio for this btw?

No. I was giving it some more thought to make the ticket more concrete

Choose a reason for hiding this comment

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

I saw this a bit too late. I created https://github.com/iterative/studio/issues/5001, but feel free to add anything you think would help clarify further.

@daavoo daavoo changed the title Add example-get-started-cv Add example-get-started-experiments Feb 1, 2023
@daavoo daavoo merged commit 20facbc into master Feb 1, 2023
@daavoo daavoo deleted the example-get-started-cv branch February 1, 2023 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants