Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

first import bpe #27

Closed
wants to merge 4 commits into from
Closed

first import bpe #27

wants to merge 4 commits into from

Conversation

DYCSystran
Copy link
Contributor

No description provided.

@srush
Copy link
Contributor

srush commented Dec 22, 2016

Please fix the luacheck errors

Checking tools/utils/BPE.lua 24 warnings
tools/utils/BPE.lua:12:5: setting non-standard global variable l
tools/utils/BPE.lua:13:10: accessing undefined variable l
tools/utils/BPE.lua:32:7: unused loop variable v
tools/utils/BPE.lua:40:3: setting non-standard global variable mintmp
tools/utils/BPE.lua:41:3: setting non-standard global variable minpair
tools/utils/BPE.lua:43:5: setting non-standard global variable pair_cur
tools/utils/BPE.lua:44:20: accessing undefined variable pair_cur
tools/utils/BPE.lua:45:7: setting non-standard global variable scoretmp
tools/utils/BPE.lua:45:29: accessing undefined variable pair_cur
tools/utils/BPE.lua:46:11: accessing undefined variable scoretmp
tools/utils/BPE.lua:46:22: accessing undefined variable mintmp
tools/utils/BPE.lua:47:9: setting non-standard global variable mintmp
tools/utils/BPE.lua:47:18: accessing undefined variable scoretmp
tools/utils/BPE.lua:48:9: setting non-standard global variable minpair
tools/utils/BPE.lua:48:19: accessing undefined variable pair_cur
tools/utils/BPE.lua:52:10: accessing undefined variable minpair
tools/utils/BPE.lua:56:9: unused variable nextv
tools/utils/BPE.lua:70:9: unused loop variable ii
tools/utils/BPE.lua:98:3: setting non-standard global variable output
tools/utils/BPE.lua:100:5: setting non-standard global variable output
tools/utils/BPE.lua:100:14: accessing undefined variable output
tools/utils/BPE.lua:102:3: setting non-standard global variable output
tools/utils/BPE.lua:102:12: accessing undefined variable output
tools/utils/BPE.lua:103:10: accessing undefined variable output

@srush srush mentioned this pull request Dec 22, 2016
@DYCSystran DYCSystran closed this Dec 23, 2016
@srush
Copy link
Contributor

srush commented Dec 23, 2016

Why is this closed? This looked close

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants