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 the workflow notebook #6

Merged
merged 3 commits into from
Mar 28, 2019
Merged

Conversation

wlandau
Copy link
Contributor

@wlandau wlandau commented Mar 27, 2019

I have enjoyed diving into your drake-powered workflow, and I am learning a lot about recipes and keras in the process. In this PR, I propose a modified workflow that trains multiple models and compares them in terms of performance. Advantages of the new R Markdown document:

  1. It motivates the value that drake brings to a deep learning workflow. You can train multiple models in parallel (see the tips at the bottom) and skip models that are already up to date.
  2. The new drake plan is easier to read, and it has more room for exploring multiple models, which is what plans were designed to do.
  3. We duplicate less data in the new plan.
  4. The new workflow serializes and unserializes the models according to keras best practices while preserving most of the convenience of drake's caching system. That way, make() can run post-processing on models trained in a previous session.

cc @edgararuiz

@wlandau
Copy link
Contributor Author

wlandau commented Mar 27, 2019

By the way, this PR fixes #4 and #5.

@edgararuiz-zz edgararuiz-zz merged commit 7e0a686 into sol-eng:master Mar 28, 2019
@edgararuiz-zz
Copy link

Awesome work, thank you Will!

wlandau-lilly added a commit to wlandau/tensorflow-w-r that referenced this pull request Mar 28, 2019
@wlandau wlandau mentioned this pull request Mar 28, 2019
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

Successfully merging this pull request may close these issues.

3 participants