-
Notifications
You must be signed in to change notification settings - Fork 175
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
Convert step-by-step stats examples to use new Job API #2842
Conversation
/build |
3d02169
to
ab8743f
Compare
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.
Mostly good, added some comments
examples/hello-world/step-by-step/cifar10/stats/image_stats.ipynb
Outdated
Show resolved
Hide resolved
examples/hello-world/step-by-step/higgs/stats/tabular_stats.ipynb
Outdated
Show resolved
Hide resolved
e8af802
to
0a36cd6
Compare
/build |
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.
PR LGTM, we might need to improve our docs
forgot to the change the real_id part, the documentation will be in separate PR, I will do one more update |
bfd8491
to
af0c4e5
Compare
2. convert df_stats from job template to job API
2) update both higgs and cifar10 stats using the new StatsJob to streamline the notebooks
af0c4e5
to
6b9a1ea
Compare
/build |
Description
A few sentences describing the changes proposed in this pull request.
Types of changes
./runtest.sh
.