Skip to content

Check caffe tool runs #1926

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
Merged
longjon merged 1 commit into from
Feb 24, 2015
Merged

Check caffe tool runs #1926

longjon merged 1 commit into from
Feb 24, 2015

Conversation

@shelhamer
Copy link
Member

@shelhamer shelhamer commented Feb 21, 2015

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