-
Notifications
You must be signed in to change notification settings - Fork 31.1k
Add missing token classification for XLM #3277
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
Add missing token classification for XLM #3277
Conversation
sshleifer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @LysandreJik
a455a4b to
24c48b5
Compare
LysandreJik
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, thanks @sakares! I rebased on your branch and added a commit for tests that should be slow (question answering and token classification in auto models)
The current modeling_xlm.py did not have alike
ForTokenClassificationclass like others, which helps for NER task comparison across all existing models.now
XLMForTokenClassificationcan be called via: