Zifu Wang comments

Results 5 comments of


                                            Zifu Wang

Modify Dice, Jaccard and Tversky losses

Hi @ericspod, thank you for reviewing my code. While adding new loss functions as separate .py files could be a workaround, my concern is that this approach would lead to...

Modify Dice, Jaccard and Tversky losses

Hi @ericspod, I've created `losses/utils.py` and put a helper function that is shared by both `dice.py` and `tversky.py`. Unit tests pass, but `mypy` tests fail. This seems related to #8149...

Modify Dice, Jaccard and Tversky losses

Hi @ericspod, all CICD tests pass. @KumoLiu, thanks for the commit.

Modify Dice, Jaccard and Tversky losses

Hi @ericspod, sorry for the late response. I remove the costly check and modify the description of `soft_label` as you suggested. I also add some test cases. When `input` is...

Jaccard, Dice and Tversky losses are incompatible with soft labels

L1 and L2 norms will yield different versions of the loss functions. The issue exists for the L1 version, but not for the L2 version. However, the L1 version is...