composer icon indicating copy to clipboard operation
composer copied to clipboard

Published 20 hours ago • verified publisher icon mosaicml

Merge `LanguageCrossEntropyLoss` and `CrossEntropyLoss` metrics

Open Landanjs opened this issue 3 years ago • 1 comments

LanguageCrossEntropyLoss metric accepts both a mapping and tensor in update(), but the remaining functionality appears the same as CrossEntropyLoss. These metrics should be combined to reduce redundant code.

Landanjs avatar Feb 01 '22 01:02 Landanjs

😬 Sorry for this, it's my code!

I can try to work on this soon -- but feel free to take a stab at it if I don't get it in time

moinnadeem avatar Feb 01 '22 04:02 moinnadeem

Closing as done

mvpatel2000 avatar Nov 03 '22 04:11 mvpatel2000