-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Support for custom architectures for the TextClassifier class #604
Comments
Sorry I can't answer your question, but where did you find that the TextClassifier only uses a simple linear layer? |
Hello @tkon3 this is a really good question. We are currently looking into refactoring some parts of Flair such as the However at this point we don't yet know how exactly this will look like. And since many of us are on vacation until beginning of April, real development of these ideas will begin only then. But yes, generally we want to make it possible for users to do this and perhaps your idea with passing a list of layers could work. Hope this answers the question somewhat - as the ideas develop, we'll keep you posted! |
Hi @NielsRogge the The self.decoder is simply a linear layer, see: However, this does not mean that the classifier is only a linear layer, since the choice of |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Hello,
I didn't find a way to use a custom architecture to classify texts so I guess it is not implemented yet.
Currently the TextClassifier class only uses a simple linear layer (nn.Linear) in order to do the task (+sigmoid).
Is allowing custom pytorch layers planned in the futur ?
Something allowing us to specify a custom architecture to the TextClassifier class with torch Sequential or a list :
The text was updated successfully, but these errors were encountered: