Get started: better dataset, dvclive, more plots #114
Conversation
|
Some things will be uncommented and reverted after data registry update lands. |
Co-authored-by: David de la Iglesia Castro <daviddelaiglesiacastro@gmail.com>
Co-authored-by: David de la Iglesia Castro <daviddelaiglesiacastro@gmail.com>
| live.log_plot("confusion_matrix", labels.squeeze(), predictions_by_class.argmax(-1)) | ||
|
|
||
| # ... and finally, we can dump an image, it's also supported: | ||
| fig, axes = plt.subplots(dpi=100) |
There was a problem hiding this comment.
@shcheklein Reduced size (from 800 to 100) here as it was very disproportionated in dvc plots show.
There was a problem hiding this comment.
@daavoo yep, I think we should also handle gracefully this on dvc plots show - images could be large after all, we should be normalizing their sizes
There was a problem hiding this comment.
and for VS Code it's better to have them bigger (you can click and zoom)
example-get-started/deploy.sh
Outdated
| @@ -17,7 +17,9 @@ popd | |||
|
|
|||
| # Requires AWS CLI and write access to `s3://dvc-public/code/get-started/`. | |||
| mv $PACKAGE_DIR/$PACKAGE . | |||
| aws s3 cp --acl public-read $PACKAGE s3://dvc-public/code/get-started/$PACKAGE | |||
| #aws s3 cp --acl public-read $PACKAGE s3://dvc-public/code/get-started/$PACKAGE | |||
There was a problem hiding this comment.
Is this intentionally left like this?
There was a problem hiding this comment.
I get it for now, just making sure we don't forget to update it.
There was a problem hiding this comment.
yep, I'll remove it today before the merge
example-get-started/generate.sh
Outdated
| --desc "Imported raw data (tracks source updates)" | ||
| git add data/data.xml.dvc | ||
| tick | ||
| git commit -m "Import raw data (overwrite)" | ||
| git tag -a "4-import-data" -m "Data file overwritten with an import." | ||
| dvc push | ||
|
|
||
| wget https://code.dvc.org/get-started/code.zip | ||
| cp $HERE/code.zip . | ||
| #wget https://code.dvc.org/get-started/code.zip |
There was a problem hiding this comment.
Again, just making sure we don't forget to uncomment.
Based on initial work done by @daavoo in this PR - #102
Depends on iterative/dataset-registry#28. A few things will be improved here when that PR lands.
Corresponding repo to play with: https://studio.iterative.ai/team/Iterative/views/example-get-started-zde16i6c4g
(a few lines in the PR will be uncommented / reverted before actual merge, after testing)
It changes example get started: