Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maybe there are some bugs due to bugs fixed of docopt #6

Open
t6am3 opened this issue Nov 7, 2019 · 0 comments
Open

Maybe there are some bugs due to bugs fixed of docopt #6

t6am3 opened this issue Nov 7, 2019 · 0 comments

Comments

@t6am3
Copy link

t6am3 commented Nov 7, 2019

In a4, we use docopt to parse our options and arguments, but if you try sh run.sh test now, you will recognize that the usage run.py decode [options] MODEL_PATH TEST_SOURCE_FILE TEST_TARGET_FILE OUTPUT_FILE doenst work.

Actually, in script run.sh, our command is python run.py decode model.bin ./en_es_data/test.es ./en_es_data/test.en outputs/test_outputs.txt --cuda and the option --cuda wont get parsing because you have not defined in your correspond usage, so we have to add an option beyond this decode usage like this:
run.py decode [options] MODEL_PATH TEST_SOURCE_FILE TEST_TARGET_FILE OUTPUT_FILE [--cuda].
Btw the options ahead just make trouble for us and if it exits we cant get the whole usage parsed, i suggest remove it and final version is as below:
run.py decode MODEL_PATH TEST_SOURCE_FILE TEST_TARGET_FILE OUTPUT_FILE [--cuda].

On an other hand, the first train usage also cant recognize the option --cuda just like the wildcard option [options] fails, so i have to modified the source code to do training, it's really annoying.

Maybe these bugs just occured after some other bugs of docopt fixed, for convience of some rookies, you can test your code and modified some content to make it work, thanks a lot!

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

No branches or pull requests

1 participant