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

Fix evaluation and other minor issues to adapt to multi-label classification #1595

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

callzhang
Copy link
Contributor

No description provided.

@callzhang callzhang changed the title Fix evaluation and other minor bug to adapt multi-label classification Fix evaluation and other minor issues to adapt to multi-label classification Dec 22, 2020
@callzhang
Copy link
Contributor Author

Tried to resolve the conflicts but it seems that you have changed the structure a lot. Any suggestion on how to merge?

@hankcs
Copy link
Owner

hankcs commented Jan 1, 2021

Yes, I refactored a lot. Basically, TensorFlow components are renamed to its original name with a TF shuffix. Sorry you have to do a line-by-line merge as I did for merging your commits to the new release. If you'd like to contribute, please rebase on dev.

By the way, since 2.1 officially dependents on the wonderful huggingface transformers. It would also be great if you want to use their BERT. Here is some reference codes: 1fe90f7

@callzhang
Copy link
Contributor Author

I eventually wrote my own script on multi-label classification task. Basically using customized BCE with weights to deal with imbalanced classes and macro-F1 for metrics, as well as AdamW with amsGrad enabled. Aided with data augmentation, the performance achieved better. I will continue to use hanlp as an exploration tool and try to contribute when I can. Thanks!

@hankcs
Copy link
Owner

hankcs commented Jan 1, 2021

Sure, feel free to explore 2.1.

@hankcs hankcs force-pushed the master branch 2 times, most recently from 3be983d to 9b3a786 Compare April 15, 2022 16:49
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.

2 participants