Skip to content

Expose more hyperparameters to FLAGS#595

Closed
djdongjin wants to merge 2 commits intotensorflow:masterfrom
djdongjin:args
Closed

Expose more hyperparameters to FLAGS#595
djdongjin wants to merge 2 commits intotensorflow:masterfrom
djdongjin:args

Conversation

@djdongjin
Copy link
Contributor

Fix TODOs in the files to expose hyperparams to FLAGS such that users can change them if needed. The default values are the same.

@kempy
Copy link
Collaborator

kempy commented May 27, 2020

Thanks this looks great! I've sent this through our internal review system and will close the PR once it's been submitted

kempy pushed a commit that referenced this pull request Jun 10, 2020
@kempy
Copy link
Collaborator

kempy commented Jun 10, 2020

Thanks! Your contribution was merged into master in 7708d8b

@kempy kempy closed this Jun 10, 2020
@kempy kempy added the merged label Jun 10, 2020
@djdongjin djdongjin deleted the args branch June 11, 2020 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants