AMR-multiview icon indicating copy to clipboard operation
AMR-multiview copied to clipboard

Some unreasonable mistakes

Open xdqkid opened this issue 4 years ago • 1 comments

model/reconstructor/utils/loss.py line 157 162.

sentence normalization token num is not equal to amr normalization token num, they should not be shared normalization.

So, in fact, ratio is uncertain.

xdqkid avatar Aug 02 '20 07:08 xdqkid

model/reconstructor/utils/loss.py line 157 162.

sentence normalization token num is not equal to amr normalization token num, they should not be shared normalization.

So, in fact, ratio is uncertain.

@xdqkid Thank you for your correction. I'll fix the code later to see if it affects the hyperparameters

Soistesimmer avatar Aug 06 '20 06:08 Soistesimmer