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.

Sign up for GitHub

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

Jump to bottom

Check caffe tool runs #1926

Merged
merged 1 commit into from
Feb 24, 2015
Merged

Check caffe tool runs #1926

merged 1 commit into from
Feb 24, 2015

Conversation

shelhamer
Copy link
Member

Simply run the caffe tool in runtest to catch obvious issues like the linking
problem fixed in #1921.

@longjon
Copy link
Contributor

longjon commented Feb 24, 2015

One thing to keep in mind here: this means caffe needs to return exit status zero when called with no arguments, rather than considering that an error condition. Given this patch we'll definitely notice if we change that, so... merging, thanks.

longjon added a commit that referenced this pull request Feb 24, 2015
@longjon
Merge pull request #1926 from shelhamer/test-caffe-tool
360dbfd 
Check caffe tool runs
@longjon longjon merged commit 360dbfd into BVLC:master Feb 24, 2015
@shelhamer shelhamer deleted the test-caffe-tool branch April 15, 2015 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shelhamer @longjon