Skip to content

Conversation

@leemgs
Copy link
Contributor

@leemgs leemgs commented Jun 29, 2017

@Noiredd
Copy link
Member

Noiredd commented Jun 29, 2017

👍, I remember having to hack a fix like that into every caffe installation in order to run my models. Would be really great to see it in stock.

@goodloop
Copy link

goodloop commented Sep 5, 2017

any problem with this pull request? why it have not been merged to master?

@Noiredd Noiredd added the focus label Oct 13, 2017
@Noiredd
Copy link
Member

Noiredd commented Oct 13, 2017

The only thing I don't like is the print statement - when bulk-testing multiple nets with their own transformers it generates a lot of doubtfully useful output, which tended to be rather annoying. But other than that, do we have anything this PR?

@leemgs
Copy link
Contributor Author

leemgs commented Oct 30, 2017

@Yangqing, @jeffdonahue, @shelhamer, Please, have a look at this.

@Noiredd
Copy link
Member

Noiredd commented Nov 2, 2017

@leemgs Please remove the print line and I will merge this.

@leemgs leemgs force-pushed the upstream-issue5718 branch from 83f5482 to bfc638d Compare November 2, 2017 14:51
@leemgs
Copy link
Contributor Author

leemgs commented Nov 2, 2017

@Noiredd I have removed the print line. Then resubmitted PR with squash.

@Noiredd
Copy link
Member

Noiredd commented Nov 6, 2017

Thank you for the fix @leemgs!

@Noiredd Noiredd merged commit 613e13b into BVLC:master Nov 6, 2017
XinYao1994 pushed a commit to XinYao1994/caffe that referenced this pull request Aug 29, 2018
Fix: mean shape incompatible with input shape
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.

4 participants