Skip to content
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

removed 4dimensions constraints on blob shape #3325

Closed
wants to merge 2 commits into from
Closed

removed 4dimensions constraints on blob shape #3325

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d93279c
removed 4dimensions constraints on blob shape
albenoit Nov 12, 2015
8d11ea9
removed 4dimensions constraints on blob shape
albenoit Nov 12, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 3 additions & 5 deletions include/caffe/blob.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,12 +137,10 @@ class Blob {
/// @brief Deprecated legacy shape accessor width: use shape(3) instead.
inline int width() const { return LegacyShape(3); }
inline int LegacyShape(int index) const {
CHECK_LE(num_axes(), 4)
<< "Cannot use legacy accessors on Blobs with > 4 axes.";
CHECK_LT(index, 4);
CHECK_GE(index, -4);
if (index >= num_axes() || index < -num_axes()) {
// Axis is out of range, but still in [0, 3] (or [-4, -1] for reverse
// Assuming num_axes is the blob dimension (generaly 4),
// axis is out of range, but still in [0, num_axes[
// (or [-num_axes, -1] for reverse
// indexing) -- this special case simulates the one-padding used to fill
// extraneous axes of legacy blobs.
return 1;
Expand Down