Skip to content

Comments

cifar10: Fix examples by setting snapshot_format.#3011

Merged
jeffdonahue merged 1 commit intoBVLC:masterfrom
darrengarvey:fixup-cifar10-example
Sep 2, 2015
Merged

cifar10: Fix examples by setting snapshot_format.#3011
jeffdonahue merged 1 commit intoBVLC:masterfrom
darrengarvey:fixup-cifar10-example

Conversation

@darrengarvey
Copy link
Contributor

Commit 4227828 set the default binary format from HDF5 to BINARYPROTO to
fix #2885. This broke the cifar10 examples which relied on this default.

This commit specifies the snapshot_format explicitly since the rest of the
example relies on this being HDF5.

Commit 4227828 set the default binary format from HDF5 to BINARYPROTO to
fix BVLC#2885. This broke the cifar10 examples which relied on this default.

This commit specifies the snapshot_format explicitly since the rest of the
example relies on this being HDF5.
@jeffdonahue
Copy link
Contributor

Not sure why Travis isn't running here, but the example prototxts aren't part of the build/tests anyway, so I'll merge. Thanks for the fixes @darrengarvey!

jeffdonahue added a commit that referenced this pull request Sep 2, 2015
cifar10: Fix examples by setting snapshot_format.
@jeffdonahue jeffdonahue merged commit d900435 into BVLC:master Sep 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HDF5 snapshot crashes on duplicate layer names

2 participants