-
Notifications
You must be signed in to change notification settings - Fork 264
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
Matthews correlation coefficient #362
Conversation
The documentation is not available anymore as the PR was closed or merged. |
Thanks for adding this! The tests for metrics are in the docstrings itself. So if you add an example or two to the docstring the will be tested by the CI. Also to fix the style tests you can run |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
Should be all ready now @lvwerra |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good! Would you mind also updating the README with the new options?
Co-authored-by: Leandro von Werra <lvwerra@users.noreply.github.com>
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
@lvwerra gave it a shot, but there's not a lot of examples to go on as the multilabel config option is quite under-documented in general. |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
Thank you for adding this! |
See #317