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

Add tutorial and fix issues for NER #2232

Merged
merged 7 commits into from
Oct 21, 2022
Merged

Add tutorial and fix issues for NER #2232

merged 7 commits into from
Oct 21, 2022

Conversation

cheungdaven
Copy link
Contributor

@cheungdaven cheungdaven commented Oct 19, 2022 •
edited
Loading

Description of changes:

  1. Add a tutorial for NER.
  2. The seqeval lib will remove the first two characters for non-BIO format and cause non-readable result report, issue. We fix it be adding prefix 'b-' or 'i-'.
  3. Fix output shape inconsistency in continuous training.
  4. Convert labels/metrics to lowercase.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link

Job PR-2232-bfcee65 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/bfcee65/index.html

@github-actions
Copy link

Job PR-2232-efbf1d5 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/efbf1d5/index.html

@github-actions
Copy link

Job PR-2232-d8923d5 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/d8923d5/index.html

@cheungdaven cheungdaven changed the title Fix issues in NER Add tutorial and fix issues for NER Oct 20, 2022
@github-actions
Copy link

Job PR-2232-ecaae4f is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/ecaae4f/index.html

@github-actions
Copy link

Job PR-2232-94efd54 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/94efd54/index.html

@github-actions
Copy link

Job PR-2232-c71ce03 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2232/c71ce03/index.html

sxjscience
sxjscience approved these changes Oct 21, 2022
View reviewed changes
Copy link
Collaborator

@sxjscience sxjscience left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!!

bryanyzhu
bryanyzhu approved these changes Oct 21, 2022
View reviewed changes
Copy link
Contributor

@bryanyzhu bryanyzhu left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@sxjscience sxjscience merged commit eafe57d into autogluon:master Oct 21, 2022
@cheungdaven cheungdaven deleted the ner branch November 30, 2022 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhiqiangdon zhiqiangdon zhiqiangdon left review comments

@sxjscience sxjscience sxjscience approved these changes

@bryanyzhu bryanyzhu bryanyzhu approved these changes

@FANGAreNotGnu FANGAreNotGnu Awaiting requested review from FANGAreNotGnu

@suzhoum suzhoum Awaiting requested review from suzhoum

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

Successfully merging this pull request may close these issues.

None yet
4 participants
@cheungdaven @sxjscience @bryanyzhu @zhiqiangdon