-
Notifications
You must be signed in to change notification settings - Fork 935
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
Add pytorch-ignite with test #226
Conversation
Thanks! I'll be testing this out today.
…On Thu, Jun 21, 2018, 6:03 AM vfdev ***@***.***> wrote:
Adds ignite recently released library to help with training neural
networks in PyTorch
More info here <https://pytorch.org/ignite/>
- docker image built and tested as described in README
------------------------------
You can view, comment on, or merge this pull request online at:
#226
Commit Summary
- Add ignite with test
File Changes
- *M* Dockerfile
<https://github.com/Kaggle/docker-python/pull/226/files#diff-0> (1)
- *M* test_build.py
<https://github.com/Kaggle/docker-python/pull/226/files#diff-1> (4)
Patch Links:
- https://github.com/Kaggle/docker-python/pull/226.patch
- https://github.com/Kaggle/docker-python/pull/226.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#226>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIBDxQT7POdA6h4M2sp5NGe_xl2X-ckOks5t-5m0gaJpZM4UyBo7>
.
|
@crawforc3 I already tested it as it is described in the README:
Everything goes correctly I would say, here is output of click
|
Awesome, thanks. I still need to test it on our end before I push it into
production is all
…On Sat, Jun 23, 2018, 6:59 AM vfdev ***@***.***> wrote:
@crawforc3 <https://github.com/crawforc3> I already tested it as it is
described in the README:
- I ran the script ./build
- I ran the script ./tests
Everything goes correctly I would say, here is output of ./test:
click
docker-python$ ./test
Error: No such container: jupyter_test
123222a493665d097ab3d3e10f32f62881f65c4fb6019bcfe7dab47e72d3df65
jupyter_test
jupyter_test
1.9.0-rc0
b'TensorFlow ok'
Tensorflow ok
Using TensorFlow backend.
Keras ok
tensor([[ 0.8517, -0.6919, 0.2332],
[ 0.3050, -0.8079, -0.3354]])
PyTorch okfast.ai ok
ignite ok
Numpy imported ok
Your lucky number is: 27
Pandas imported ok
sklearn imported ok
sklearn RandomForestClassifier: ok
sklearn LinearRegression: ok
xgboost XGBClassifier: ok
matplotlib.pyplot ok
Basemap ok
plotly ok
Theano ok
nltk ok
OpenCV ok
skimage ok
wordbatch ok
pyfasttext ok
fastText ok
mxnet ok
bokeh ok
seaborn ok
127.0.0.1 - - [23/Jun/2018 13:56:23] "GET /bigquery/v2/projects/test/datasets HTTP/1.1" 200 -
bigquery proxy ok
shap ok
kmapper ok
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#226 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIBDxVCVYdVahXwa3jV5yEJtE3ZkOl0uks5t_knfgaJpZM4UyBo7>
.
|
@crawforc3 any updates on this ? |
@vfdev-5 Hey, I'm also missing ignite in kaggle kernels... Why did you close that ticket? Is it solved somewhere else? |
@aprotopopov I have no idea why there were no actions on the PR since 06/2018... Anyway, sinice there is internet available in kernels, I install it with |
Yes, but in current jigsaw challenge internet is restricted. I added pytorch wheel 0.2.0 as kaggle dataset for now. But I hope that ignite will be introduced later by default. |
@vfdev-5 Could you reopen issue? Or maybe we could write somewhere else about supporting pytorch-ignite.. |
cc @aprotopopov |
@Philmod could you please review and give a feedback whether this PR can be merged ? Thanks |
@vfdev-5 Can you merge the changes from master into your branch? Otherwise, I tested your change and it looks good :-) |
@Philmod I merged the master, please check if it is OK for you. |
Adds
ignite
recently released library to help with training neural networks in PyTorchMore info here