Skip to content

Conversation

@netheril96
Copy link
Contributor

This is an attempt to clean up and improve upon @sguada work in #523. It is a work in progress, not ready for merge yet.

It assumes labels are a vector of {-1,0,1} with -1 for negative, 1 for positive and 0 ignore

Conflicts:

	src/caffe/proto/caffe.proto
netheril96 and others added 5 commits October 30, 2014 16:59
Conflicts:

	src/caffe/layers/multi_label_accuracy_layer.cpp
	src/caffe/test/test_multi_label_accuracy_layer.cpp

Forgot to fill the precision and f1-score in top

Conflicts:

	src/caffe/layers/multi_label_accuracy_layer.cpp
Original source: dae5300 by sguada

The merge conflicts are too many, so the original code is adapted and
inserted manually onto the current dev branch.
@bhack
Copy link
Contributor

bhack commented Nov 6, 2014

With @mtamburrano we have starter to test this also merging your indirect layer branch. Do you still want to use the same sigmoid cross entropy changes done in @sguada PR?

@netheril96
Copy link
Contributor Author

@bhack I am going to finish the implementation of IndirectionLayer in a few days and submit it as a separate pull request. It is useful regardless of this MultiLabel support. For example, it can be used in conjunction with EuclideanLossLayer or SigmoidCrossEntropyLossLayer.

As for the @sguada PR, I have already cleaned up the core of it in this branch, minus the part that would be rendered obsolete by IndirectionLayer. But I haven't come to a definite conclusion whether it is worth pursuing further, given the apparent similarity with SigmoidCrossEntropyLossLayer.

@UriChan
Copy link

UriChan commented Nov 18, 2014

@bhack , @sguada ,recently i am trying to implement the work in Street View House Number (pag. 13 http://arxiv.org/pdf/1312.6082v4.pdf), and i have a question, how can we write a global loss, and back propogate the loss the the multiple softmax layer? thanks

@sunbaigui
Copy link

I've followed #523 to here, I'm wondering when will this be merged into dev or master?

@elaksana
Copy link

Same here, waiting for this to be merged.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this assignment outside for loop?

@immars
Copy link

immars commented Feb 4, 2015

@netheril96 can you provide a example net definition showing how to use multi-label loss layers?
As Datum.label is not repeated, is it correct that I introduce a separate data layer for label vectors?

@misaghkord
Copy link

when i try to built this pr i get the following error
make: *** [.build_release/src/caffe/layers/multi_label_accuracy_layer.o] Error 1
can anyone help me please?

@shelhamer shelhamer added the ES label Mar 10, 2015
@danielhauagge
Copy link

@netheril96, has this been abandoned?

@bhack
Copy link
Contributor

bhack commented Mar 29, 2015

@danielhauagge I think that core developers want to declare this solved with #1698. So I think that will be hard to have multi label support from text file like with this PR.

@sayanghosh
Copy link

I have succesfully implemented multilabel support along with a soft max loss function. Will plan to submit a PR after ICCV deadline. I don't know why multi-label doesn't have support, it being a potentially important feature. @netheril96 @bhack

@netheril96
Copy link
Contributor Author

@danielhamngren

Yes, I abandoned this PR.

@netheril96 netheril96 closed this Mar 30, 2015
@gavinmh
Copy link

gavinmh commented Apr 20, 2015

@sayanghosh can you share your work on multilabel support?
@bhack can you explain how this feature is considered complete and why this PR is being abandoned?

@samratkokula samratkokula mentioned this pull request Dec 11, 2015
cypof pushed a commit that referenced this pull request Jun 13, 2017
cypof pushed a commit that referenced this pull request Jun 13, 2017
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.