Repository navigation
Added dice loss - #224
Added dice loss#224ayyucedemirbas wants to merge 2 commits into
Conversation
awni
left a comment
There was a problem hiding this comment.
Thanks for the contribution!!
This loss seems a little niche to add it to mlx.nn.losses. I'm open to discussion on it, but unless there is a good case / strong need, I would table this addition for now and bring it back if it's useful.
|
Thank you so much for your time and quick response. It can be useful for image segmentation tasks. But the final decision is yours 🙂 I would like to add other things (losses, layers, activation functions, etc.), but I don't know what would be useful. I think a list of things to add could be helpful for the contributors. |
|
Great. Let's close this for now. And we can revisit in the futre. A list of open items that would make for good contributions is a nice idea :). In the meantime, a good place to start is to check out the issues an look for the label enhancements. Feel free to post on the issue if you have questions about the feature. |
Proposed changes
Please include a description of the problem or feature this PR is addressing. If there is a corresponding issue, include the issue #.
Checklist
Put an
xin the boxes that apply.pre-commit run --all-filesto format my code / installed pre-commit prior to committing changes