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

Add support for function_to_apply in classification #211

Merged
merged 5 commits into from
May 23, 2023
Merged

Add support for function_to_apply in classification #211

merged 5 commits into from
May 23, 2023

Conversation

coderrg
Copy link
Contributor

@coderrg coderrg commented May 20, 2023

For classification models, users should have some way of specifying the function to be applied to the logits rather than softmax always being used since the logits do not always correspond to mutually exclusive classes.

Take, for example, a model's logits representing whether a piece of text conveys negative, neutral, or positive sentiment in a text classification task. In this case, the softmax activation function is apt as it ensures that the probabilities in the final output vector sum to 1. However, if the logits denote whether the text is toxic, obscene, insulting, etc. (as seen in Detoxify models), the sigmoid activation function would be more appropriate since the values in the final output vector do not necessarily have to sum to 1.

HF pipelines used softmax as the mandatory activation function over multi-class model logits until huggingface/transformers#8328, when they added support for the function_to_apply parameter which can be specified by the user.

I am proposing support for a function_to_apply parameter here as well. The options are the same as implemented in HF: softmax, sigmoid, and none (none applies no activation function and just returns the raw logits). softmax is the default since it preserves backward compatibility and seems to be the most common case.

@coderrg
Copy link
Contributor Author

coderrg commented May 23, 2023

Thank you for your feedback, @jonatanklosko! I have implemented your suggested changes.

Copy link
Member

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

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

Awesome, thanks :)

@jonatanklosko jonatanklosko merged commit b9acb4a into elixir-nx:main May 23, 2023
1 check passed
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.

None yet

2 participants