man-sf-emnlp icon indicating copy to clipboard operation
man-sf-emnlp copied to clipboard

potential bug

Open dotchen opened this issue 2 years ago • 0 comments

Here https://github.com/midas-research/man-sf-emnlp/blob/main/train.py#L107-L109,

Why is this F.softmax? should the right thing be F.log_softmax?

reference: https://pytorch.org/docs/1.9.0/generated/torch.nn.functional.nll_loss.html

dotchen avatar Dec 06 '21 03:12 dotchen