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 widget to copy python file from client or server to job dir #329

Merged
merged 1 commit into from
Nov 4, 2015

Conversation

jmancewicz
Copy link
Contributor

Copies the python file to the job directory from the server of the client.

@lukeyeager
Copy link
Member

Cool stuff! Can you post an example of how to use this?

@jmancewicz
Copy link
Contributor Author

Sure. I'll try to make a simplified example. Where do I post examples?

@lukeyeager
Copy link
Member

Where do I post examples?

Just to this thread is fine for now.

@jmancewicz jmancewicz force-pushed the python-layer branch 2 times, most recently from 3f70b23 to d9c9a6d Compare October 9, 2015 20:07
@jmancewicz
Copy link
Contributor Author

Added a python layer test. @lukeyeager please take a look.

@lukeyeager
Copy link
Member

Uh oh, the TravisCI build is failing with:

ERROR: Train Caffe Model: Check failed: registry.count(type) == 1 (0 vs. 1) Unknown layer type: Python (known types: AbsVal, Accuracy, ArgMax, BNLL, Concat, ContrastiveLoss, Convolution, Data, Deconvolution, Dropout, DummyData, Eltwise, EuclideanLoss, Exp, Flatten, HDF5Data, HDF5Output, HingeLoss, Im2col, ImageData, InfogainLoss, InnerProduct, LRN, MVN, MemoryData, MultinomialLogisticLoss, PReLU, Pooling, Power, ReLU, Reshape, SPP, Sigmoid, SigmoidCrossEntropyLoss, Silence, Slice, Softmax, SoftmaxWithLoss, Split, TanH, Threshold, WindowData)

Looks like we need to add this line:

sed -i 's/# WITH_PYTHON_LAYER/WITH_PYTHON_LAYER/g' Makefile.config

To this file in order to get Caffe building with Python layers on Travis.

@jmancewicz jmancewicz force-pushed the python-layer branch 5 times, most recently from d49340e to 2bfce2d Compare October 23, 2015 23:08
@jmancewicz jmancewicz force-pushed the python-layer branch 7 times, most recently from 26b66e2 to be76d25 Compare November 2, 2015 23:58
@lukeyeager
Copy link
Member

@jmancewicz please remove the last commit. We'll address that in #396. Since reverting #394, you should be able to get the tests to pass here.

lukeyeager added a commit that referenced this pull request Nov 4, 2015
Add widget to copy python file from client or server to job dir
@lukeyeager lukeyeager merged commit c4cc194 into NVIDIA:master Nov 4, 2015
@lukeyeager lukeyeager mentioned this pull request Nov 17, 2015
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants