Skip to content

Comments

Adapt python net drawing to new proto format#1984

Closed
axel-angel wants to merge 1 commit intoBVLC:masterfrom
axel-angel:master
Closed

Adapt python net drawing to new proto format#1984
axel-angel wants to merge 1 commit intoBVLC:masterfrom
axel-angel:master

Conversation

@axel-angel
Copy link

The current version of draw_net.py that's using draw.py plots an empty figure on the LeNet prototxt. I've heard the format of this file changed/is changing and thus I made the adaptation that works.

Can you review this?

@shelhamer shelhamer added the JD label Feb 26, 2015
@shelhamer
Copy link
Member

@axel-angel thanks for checking this but this change is actually in reverse. The current code is right as layer type has changed to a string #1694. The examples should all be in this new format. @jeffdonahue is there old proto somewhere?

@jeffdonahue
Copy link
Contributor

No, there isn't anything left in the old format that I know of, but let me know if I'm mistaken. These changes seem to reverse @tnarihi's fixes in #1884, so I'll close this.

@axel-angel
Copy link
Author

Okay then, that would explain why I have a deprecation warning(1). I need to adapt my prototxt to the new format then, I'll look into it. I had the case where "layers" were not recognized and thus I switched to "layer" and such, and that's probably due to my repository not updated to last version (seems the modification went the 5th February). Thank you for the clarification!

(1)

  • E0226 11:58:30.210986 10845 upgrade_proto.cpp:618] Attempting to upgrade input file specified using deprecated V1LayerParameter: snapshots/lenet_mnist_v2_iter_10000.caffemodel
  • I0226 11:58:30.211860 10845 upgrade_proto.cpp:626] Successfully upgraded file specified using deprecated V1LayerParameter

@tnarihi
Copy link
Contributor

tnarihi commented Feb 26, 2015

The problem is the protobuf itself does not support automatically updating old format. Should we add something to prohibit to draw old format protofile? I think no message is not good. Otherwise, merge layrers and layer after reading net proto?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants