Jason Senthil comments

Repositories
Issues
Comments

Results 4 comments of


                                            Jason Senthil

Fix/syncmode

> should I change Parallel.SyncMode to Parallel.StateSyncMode?

Multiple metrics sharing the same state

Hi @ZhiyuanChen, this makes sense to us! Feel free to make a pull request!

fix: correct reference length calculation

Hi @yuxqiu, thanks for this contribution! it seems some files unrelated to BLEU have been formatted in a way in which causes our linter to error, do you mind undo-ing...

fix: correct reference length calculation

Hi @yuxqiu, thanks for reverting! We have identified the linter issue to be on our end, we'll land a fix first then rerun these tests again :)