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

Decompose Trainer.train() into smaller methods #204

Closed
matt-gardner opened this issue Aug 27, 2017 · 0 comments
Closed

Decompose Trainer.train() into smaller methods #204

matt-gardner opened this issue Aug 27, 2017 · 0 comments

Comments

@matt-gardner
Copy link
Contributor

It's grown to be a ~150 line method that's pretty hard to reason about. It's a bit tricky to decompose, though, because of all of the dependencies between different parts of the method, but we should be able to pull out a bunch of it into separate methods.

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

1 participant