Skip to content
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

Merged
merged 3 commits into from
May 21, 2019
Merged

Add pytorch-ignite with test #226

merged 3 commits into from
May 21, 2019

Conversation

vfdev-5
Copy link
Contributor

@vfdev-5 vfdev-5 commented Jun 21, 2018

Adds ignite recently released library to help with training neural networks in PyTorch
More info here

  • docker image built and tested as described in README

@ghost
Copy link

ghost commented Jun 22, 2018 via email

@vfdev-5
Copy link
Contributor Author

vfdev-5 commented Jun 23, 2018

@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 ok
fast.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

@ghost
Copy link

ghost commented Jun 23, 2018 via email

@vfdev-5
Copy link
Contributor Author

vfdev-5 commented Jun 29, 2018

@crawforc3 any updates on this ?

@vfdev-5 vfdev-5 closed this Feb 14, 2019
@aprotopopov
Copy link
Contributor

@vfdev-5 Hey, I'm also missing ignite in kaggle kernels... Why did you close that ticket? Is it solved somewhere else?
@crawforc3 are you going to add ignite to docker-python?

@vfdev-5
Copy link
Contributor Author

vfdev-5 commented May 10, 2019

@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 !pip install pytorch-ignite.

@aprotopopov
Copy link
Contributor

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.

@aprotopopov
Copy link
Contributor

@vfdev-5 Could you reopen issue? Or maybe we could write somewhere else about supporting pytorch-ignite..

@vfdev-5
Copy link
Contributor Author

vfdev-5 commented May 16, 2019

cc @aprotopopov

@vfdev-5 vfdev-5 reopened this May 16, 2019
@vfdev-5 vfdev-5 changed the title Add ignite with test Add pytorch-ignite with test May 16, 2019
@vfdev-5
Copy link
Contributor Author

vfdev-5 commented May 16, 2019

@Philmod could you please review and give a feedback whether this PR can be merged ? Thanks

@Philmod
Copy link
Contributor

Philmod commented May 20, 2019

@vfdev-5 Can you merge the changes from master into your branch? Otherwise, I tested your change and it looks good :-)

@vfdev-5
Copy link
Contributor Author

vfdev-5 commented May 21, 2019

@Philmod I merged the master, please check if it is OK for you.

Dockerfile Outdated Show resolved Hide resolved
@Philmod Philmod merged commit 509e370 into Kaggle:master May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants