Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

--cuda and --tied are True by default? #23

Open
sanyam5 opened this issue Feb 19, 2018 · 2 comments
Open

--cuda and --tied are True by default? #23

sanyam5 opened this issue Feb 19, 2018 · 2 comments

Comments

@sanyam5
Copy link

sanyam5 commented Feb 19, 2018

In specifying commandline arguments in main.py and finetune.py the --cuda and --tied flags are True by default and get False on specifying. Is this intentional? Seems counter-intuitive. Does this have any bearing on the results in your paper Regularizing and Optimizing LSTM Language Models?

parser.add_argument('--tied', action='store_false',

@sanyam5
Copy link
Author

sanyam5 commented Feb 24, 2018

tagging @Smerity @keskarnitish

@keskarnitish
Copy link
Contributor

Our apologies; we intend for both to be on by default so as to reduce the number of flags passed for each run. We will work on fixing that to make it --no-cuda and --no-tied or equivalent to prevent the confusion.

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

No branches or pull requests

2 participants