torchmetrics icon indicating copy to clipboard operation
torchmetrics copied to clipboard

Bugfix: ignore index + macro combination in classification metrics

Open SkafteNicki opened this issue 1 year ago • 4 comments

What does this PR do?

Fixes #2441

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--2443.org.readthedocs.build/en/2443/

SkafteNicki avatar Mar 09 '24 15:03 SkafteNicki

@SkafteNicki, what is missing here to make it land? :chipmunk:

Borda avatar May 21 '24 09:05 Borda

Is this still happening?

DimitrisMantas avatar Jun 08 '24 05:06 DimitrisMantas

Any chance that this will be worked on in the near future?

JeroenMandersloot avatar Jun 21 '24 09:06 JeroenMandersloot

Any chance that this will be worked on in the near future?

cc: @SkafteNicki

Borda avatar Jul 22 '24 07:07 Borda

Any chance that this will be worked on in the near future?

do you want to take it over and reopen @SkafteNicki commits as part of your/new PR so you will share contribution? 🦩

Borda avatar Mar 10 '25 12:03 Borda

@DimitrisMantas @JeroenMandersloot are you interested in taking this over and finishing it? 🦩

Borda avatar Mar 21 '25 07:03 Borda

@rittik9, would you be interested in finishing this one? 🐰

Borda avatar Jul 02 '25 20:07 Borda

Yeah sure I'll give it a try

rittik9 avatar Jul 02 '25 20:07 rittik9

@SkafteNicki let's finish it or close it? 🐿️

Borda avatar Jul 15 '25 13:07 Borda