Skip to content

Improve caffe tool#872

Merged
jeffdonahue merged 11 commits intoBVLC:devfrom
shelhamer:caffe-tool
Aug 7, 2014
Merged

Improve caffe tool#872
jeffdonahue merged 11 commits intoBVLC:devfrom
shelhamer:caffe-tool

Conversation

@shelhamer
Copy link
Member

Follow up on #789 to make the caffe tool more complete and friendly.

  • produce normal output by default
  • groom args: shorten, add description, and merge gpu + device_id
  • add checks
  • add usage message
  • incorporate test

Any other suggestions before release to master?

@shelhamer
Copy link
Member Author

I'm happy with this for a release. @Yangqing @jeffdonahue ?

Happy to make any style changes and squash too.

@shelhamer
Copy link
Member Author

The --gpu and --device_id are combined into a single --gpu=${DEVICE_ID} flag per @jeffdonahue's offline suggestion. Merge?

@jeffdonahue
Copy link
Contributor

Nice streamlining, thanks Evan!

jeffdonahue added a commit that referenced this pull request Aug 7, 2014
@jeffdonahue jeffdonahue merged commit 5ed0df2 into BVLC:dev Aug 7, 2014
@shelhamer shelhamer mentioned this pull request Aug 7, 2014
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
RazvanRanca pushed a commit to RazvanRanca/caffe that referenced this pull request Nov 4, 2014
@shelhamer shelhamer deleted the caffe-tool branch February 26, 2015 00:50
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

Comments