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.

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

Only check the dimensions start from start_axis #3778

Closed
wants to merge 1 commit into from
Closed

Only check the dimensions start from start_axis #3778

wants to merge 1 commit into from

Conversation

wk910930
Copy link
Contributor

@wk910930 wk910930 commented Mar 6, 2016

crop_offset may not be suitable or needed for dimensions before start_axis.

* The dimensions before start_axis is okay to have its own size but might not pass the check
@MalteOeljeklaus
Copy link

MalteOeljeklaus commented Apr 7, 2016

+1, I also need this! The present code in the master branch does not allow different numbers of channels for the two input blobs even when only the spatial dimensions are cropped. This does not affect the original FCN models but is obstructive when implementing models like in the hypercolumn paper. I've yet to do more testing but for now this pull request seems to work for me.

Edit: #3993 has been merged so this is probably obsolete

@shelhamer
Copy link
Member

shelhamer commented Apr 16, 2016

@wk910930 Sorry, you're totally right! Thanks for this and thanks @MalteOeljeklaus for confirming. This was fixed in #3993. Sorry again that this was lost in the deadline whirlwind of March.

@shelhamer shelhamer closed this Apr 16, 2016
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.

3 participants