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

[Feature]: Why SequenceTagger doesn't inherits from DefaultClassifier #3562

Open
yidengcoupang opened this issue Nov 6, 2024 · 1 comment
Labels
feature A new feature

Comments

@yidengcoupang
Copy link

Problem statement

I want to use decoder feature in SequenceTagger. And found SequenceTagger inherit directly from Classifier instead of DefaultClassifier like most of other models. So what's missing here and how can we add decoder function into SequenceTagger or let SequenceTagger inherit from DefaultClassifier?

Solution

maybe migrate SequenceTagger to inherit from DefaultClassifier.

Additional Context

No response

@yidengcoupang yidengcoupang added the feature A new feature label Nov 6, 2024
@helpmefindaname
Copy link
Collaborator

Hi @yidengcoupang
SequenceTagger doesen't inherit from DefaultClassifier, as it's usage of the CRF is more complex than the default settings. You can use the TokenClassifier for a simpler implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature
Projects
None yet
Development

No branches or pull requests

2 participants