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

Allow for arbitrary DAGs using split layers (Take 2) #129

Merged
merged 23 commits into from
Feb 19, 2014

Conversation

jeffdonahue
Copy link
Contributor

Now aware of existing in-place layers, fixing the bug from #114. (Includes a unit test checking that the actual network architecture from imagenet.prototxt is unmodified.)

@shelhamer
Copy link
Member

I re-arranged this so history is sensible: all the dags-by-splitcommits appear in order without the remerge/revert business. Bug fix starts at 0d6bc43.

shelhamer added a commit that referenced this pull request Feb 19, 2014
Generalize architectures to arbitrary DAGs by split layers
@shelhamer shelhamer merged commit c57f355 into BVLC:dev Feb 19, 2014
@shelhamer
Copy link
Member

the city of forks welcomes you

@sguada
Copy link
Contributor

sguada commented Feb 19, 2014

Nice job @jeffdonahue and nice merge @shelhamer

shelhamer added a commit to shelhamer/caffe that referenced this pull request Feb 23, 2014
Generalize architectures to arbitrary DAGs by split layers
shelhamer added a commit to shelhamer/caffe that referenced this pull request Feb 26, 2014
Generalize architectures to arbitrary DAGs by split layers
shelhamer added a commit that referenced this pull request Feb 26, 2014
Generalize architectures to arbitrary DAGs by split layers
shelhamer added a commit that referenced this pull request Feb 26, 2014
Generalize architectures to arbitrary DAGs by split layers
shelhamer added a commit that referenced this pull request Feb 26, 2014
Welcome to the model of forking paths.
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
Welcome to the model of forking paths.
borisgin pushed a commit to borisgin/caffe that referenced this pull request Apr 9, 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