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

Refactor training graph generators #15

Open
rdadolf opened this issue Sep 16, 2016 · 0 comments
Open

Refactor training graph generators #15

rdadolf opened this issue Sep 16, 2016 · 0 comments
Labels
Milestone

Comments

@rdadolf
Copy link
Owner

rdadolf commented Sep 16, 2016

The ops required for training (loss function, optimizer, and training loop) are spread out over too many files. They are difficult to read and follow.

These should be integrated into the models themselves, even if it means a little bit of duplication across files. Like the specification of the inference graphs (see #9), a reader who knows the training method behind a given model should be able to recognize it on sight.

@rdadolf rdadolf added this to the Release 1.0 milestone Sep 16, 2016
@rdadolf rdadolf added the H label Oct 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant