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

Split Prediction Heads and make Custom Heads possible #88

Merged
merged 36 commits into from
Dec 2, 2020
Merged

Split Prediction Heads and make Custom Heads possible #88

merged 36 commits into from
Dec 2, 2020

Conversation

hSterz
Copy link
Member

@hSterz hSterz commented Nov 24, 2020

Made individual modules for the prediction heads
Custom Heads can be used and saved that inheret from the PredictionHead

https://trello.com/c/e8bOC4zZ

hSterz and others added 30 commits October 16, 2020 08:30
@hSterz
Added get_labels() and get_labels_dict() for model with explicit head
0d5a370
@hSterz
Added get_labels() and get_labels_dict() for ModelWithHeads so that d…
15f0ac0 
…ifferent heads can have different labels
@hSterz
Improved save label tests
71756cc
@hSterz
Quality for tests
a13e984
@hSterz
Include review feedback
76f047e
@hSterz
Quality
5901aeb
@hSterz
Set model labels, lable2id and id2label acording to Head for BertMode…
5100780 
…lWithHeads
@hSterz
Changes from forked
8dce52d
@hSterz
Fixed accessing id2label
f9700fb
@hSterz
Fixed
3330762
@hSterz
Fixed labels
603e020
@hSterz
IMplemented with @Property
d029259
@hSterz
Adapter Head Module
accc843
@hSterz
Quality
5a58ca4
@hSterz
quality
91af39c
@hSterz
Revert "quality"
875af06 
This reverts commit 91af39c
@hSterz
quality
93e7a20
@hSterz
Quality
2535144
@hSterz
Added saving and loading of heads
7e8495b
Added Multi Label Classification Head
327d8ba
Fixed Eval mode for Head
fca46b0
@hSterz
Fixed saving path
2d01d79
@hSterz
Merge
b2f0fea
@hSterz
Some fixes added test
c435bb9
@hSterz
Style
42949ba
@hSterz
Quality
90a143f
@hSterz
Changed saving of the custom heads. Put the custom heads into the mod…
3ec6487 
…el config and added tests
@hSterz
Quality
1a133e4
@hSterz
Quality
374d3cf
@calpt
Rename test case
713f794
@calpt calpt changed the title Split Prediction Heads and make Costom Heads possible Split Prediction Heads and make Custom Heads possible Dec 2, 2020
@calpt calpt self-requested a review December 2, 2020 17:13
@calpt calpt marked this pull request as ready for review December 2, 2020 17:13
calpt
calpt approved these changes Dec 2, 2020
View reviewed changes
Copy link
Member

@calpt calpt left a comment

Choose a reason for hiding this comment

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

👍

@calpt calpt merged commit c3ed951 into adapter-hub:master Dec 2, 2020
@calpt calpt mentioned this pull request Dec 7, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calpt calpt calpt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hSterz @calpt