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

How do you train the bottom up model with attributes #5

Closed
ghost opened this issue Aug 7, 2018 · 5 comments
Closed

How do you train the bottom up model with attributes #5

ghost opened this issue Aug 7, 2018 · 5 comments

Comments

@ghost
Copy link

ghost commented Aug 7, 2018

Hi:

I want to know how you add predicting attributes in bottom up model,because i currently can't find it in repo. Would you mind providing the code for that ?

Best Wishes
Hanshan Zhang

@endernewton
Copy link
Contributor

We followed the same as the original bottom up model did: https://github.com/peteanderson80/bottom-up-attention/blob/master/models/vg/ResNet-101/faster_rcnn_end2end_final/train.prototxt. We predict the attribute with both the ground truth object class embedding and the bottom RoI features. Nothing new.

For the actual code to train the model, it is currently mixed with other efforts we current have. It will take a while to clean that up and release. But it is purely based on the Caffe2 detectron and we follow the same as Anderson et al did -- should not be hard to reproduce on your end.

@endernewton
Copy link
Contributor

IIRC, I had to fix a bug for predicting the attributes when implementing that (cannot recall what it exactly it is but it is related to the cross entropy loss). But it may work even work without that bug fixed.

@ghost
Copy link
Author

ghost commented Aug 8, 2018

Hi,

Can u recall the performance of predicting the attributes ? I want to compare it with my implementation

@endernewton
Copy link
Contributor

I did not check the prediction of attributes, I did check the detection performance though, which is in the same ballpark of what reported by Anderson.

@ghost
Copy link
Author

ghost commented Aug 8, 2018

Thanks for your help. I will look at it. Thanks again

apsdehal pushed a commit that referenced this issue Apr 23, 2019
* Add preprocessing util tests

* Add timer util tests

* Add general util tests

* Replace dict keyword with dictionary and other suggested changes.
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

No branches or pull requests

1 participant