[Bug] Multiple references

facebookresearch / vizseq

An Analysis Toolkit for Natural Language Generation (Translation, Captioning, Summarization, etc.)

https://arxiv.org/abs/1909.05424

MIT License

443 stars 59 forks source link

[Bug] Multiple references #16

Closed nadjet closed 4 years ago

nadjet commented 4 years ago

When having multiple references (attached muti_refs.zip), I cannot configure metric BLEU (but I can configure GLEU). I get a 500 internal server error.

This does not happen with a single reference (see attached single_ref.zip).

kahne commented 4 years ago

Hi @nadjet , thank you very much for reporting the issue! I made a fix in #17. Can you pull the latest code from github and reinstall VizSeq:

pip uninstall vizseq
git clone https://github.com/facebookresearch/vizseq
cd vizseq
pip install -e .

The fix will be included in 0.1.10 release as well, which will come out very soon. I will send you a notice once it's available.

kahne commented 4 years ago

@nadjet 0.1.10 has been released. To upgrade, you may want to do

pip install --upgrade vizseq

nadjet commented 4 years ago

It works now. Thank you for fixing this.