Skip to content

Conversation

jkauerl
Copy link
Contributor

@jkauerl jkauerl commented Jun 3, 2024

Pull Request Template

Description

PR including the implementation for the Negative Log Likelihood Loss Function of this issue #559 . It creates a new file called negative_log_likelihood.rs with the function called neg_log_likelihood. This implementation is for binary classifications, but can be expanded to higher dimensions.

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • I ran bellow commands using the latest version of rust nightly.
  • I ran cargo clippy --all -- -D warnings just before my last commit and fixed any issue that was found.
  • I ran cargo fmt just before my last commit.
  • I ran cargo test just before my last commit and all tests passed.
  • I added my algorithm to the corresponding mod.rs file within its own folder, and in any parent folder(s).
  • I added my algorithm to DIRECTORY.md with the correct link.
  • I checked COUNTRIBUTING.md and my code follows its guidelines.

@jkauerl jkauerl requested review from imp2002 and vil02 as code owners June 3, 2024 21:13
@codecov-commenter
Copy link

codecov-commenter commented Jun 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.06%. Comparing base (f8096d2) to head (555b3fb).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #734      +/-   ##
==========================================
+ Coverage   95.04%   95.06%   +0.01%     
==========================================
  Files         305      307       +2     
  Lines       22674    22754      +80     
==========================================
+ Hits        21550    21630      +80     
  Misses       1124     1124              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jkauerl jkauerl requested a review from vil02 June 4, 2024 15:53
@vil02
Copy link
Member

vil02 commented Jun 27, 2024

@jkauerl are you still working on that?

@jkauerl
Copy link
Contributor Author

jkauerl commented Jun 28, 2024

@jkauerl are you still working on that?

Yes, I implemented most of the suggestions, and do not know what else is missing.

@jkauerl jkauerl requested a review from vil02 July 4, 2024 10:35
@vil02 vil02 merged commit c17cb5a into TheAlgorithms:master Jul 9, 2024
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.

3 participants