-
-
Notifications
You must be signed in to change notification settings - Fork 46k
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
Added Mean Squared Logarithmic Error (MSLE) Loss Function #10637
Conversation
This directory of loss functions is less compelling than I thought because each is three lines or less of algorithmic code. The files are about 100 lines long with comments, doctests, validation code, and other boilerplate. It takes a lot of clicking in and out of files to review all of the various loss functions. Would we be better off making a single |
Sounds good to me. Once we're done handling the existing loss function PRs we can open a new PR to consolidate all of the loss functions. |
Hey so could you please accept the PR, or are there any changes I have to make? |
…hms#10637) * Added Mean Squared Logarithmic Error (MSLE) * Added Mean Squared Logarithmic Error (MSLE) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Describe your change:
I have added MSLE loss function which is very useful when dealing with regression problems involving skewed or large-value targets.
Checklist: