Skip to content

Conversation

@SkafteNicki
Copy link
Collaborator

@SkafteNicki SkafteNicki commented Feb 12, 2024

What does this PR do?

Fixes #2369

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?
PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃


📚 Documentation preview 📚: https://torchmetrics--2370.org.readthedocs.build/en/2370/

@codecov
Copy link

codecov bot commented Feb 12, 2024

Codecov Report

Merging #2370 (ea78e2d) into master (f4ef8a8) will increase coverage by 0%.
The diff coverage is 75%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #2370   +/-   ##
======================================
  Coverage      69%     69%           
======================================
  Files         305     305           
  Lines       17186   17190    +4     
======================================
+ Hits        11846   11849    +3     
- Misses       5340    5341    +1     

@mergify mergify bot added the ready label Feb 12, 2024
@Borda Borda enabled auto-merge (squash) February 13, 2024 15:05
@Borda Borda merged commit 59d0a11 into master Feb 14, 2024
@Borda Borda deleted the deprecate/kl_divergence_input_order branch February 14, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an option to switch distributions order in the KLDivergence.

3 participants