Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Prediction for example kaggle_ndsb1 #1296

Closed
wants to merge 3 commits into from

Conversation

Lopezurrutia
Copy link
Contributor

This is related to #1143 and @antinucleon example on using mxnet on Kaggle Data Science Bowl 1 image classification.

I have added prediction and the generation of submission.csv so the model output can be submitted to kaggle. I have also done minor changes to training to be in the same format as image-classification examples (maybe the whole folder could be placed inside image-classification just as another example).

@@ -0,0 +1,216 @@

Copy link
Contributor

Choose a reason for hiding this comment

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

probably we don't need this cxxnet conf file?

@Lopezurrutia
Copy link
Contributor Author

yes, sorry, I made a commit to my fork yesterday as I plan to port those files to mxnet, (I am new to github and I thought pull request would use my repository in the state when the pull request was done)
This is the repository state that should be pulled:
https://github.com/Gelu74/mxnet/tree/d9fd588d676f0be66b050b0ab76ac06f1a7359d5/example/kaggle-ndsb1
Do you want me to revert the last commit 72f8e21 in my fork?

@antinucleon
Copy link
Contributor

Thanks for your work. A simpler way is switch to a new branch, then modify commit, eg, remove files, then squash commits (https://mxnet.readthedocs.org/en/latest/contribute.html#how-to-combine-multiple-commits-into-one). After that we can send a PR. After merge it, you are safe to delete the branch, and your master branch won't be affected. Maybe we can close this PR and open a new PR from a new branch?

@Lopezurrutia
Copy link
Contributor Author

OK, close this one and I'll make a new PR in a few hours

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants