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

Workflow follow-up #7

Merged
merged 1 commit into from
Mar 28, 2019
Merged

Workflow follow-up #7

merged 1 commit into from
Mar 28, 2019

Conversation

wlandau
Copy link
Contributor

@wlandau wlandau commented Mar 28, 2019

This PR follows up on #6:

  • Convert the rendered HTML of workflow/tensorflow-drake.Rmd back to notebook form.
  • Update images/dep-graph.png.
  • Remove superfluous files such as workflow/newmodel/.

After this, I think it just remains to refresh the notebook at colorado.rstudio.com.

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

Awesome job, thank you

I just updated the notebook, please let me know if that looks ok to you: https://colorado.rstudio.com/rsc/content/2159/tensorflow-drake.nb.html

@wlandau
Copy link
Contributor Author

wlandau commented Mar 31, 2019

Thanks, @edgararuiz! I think the notebook rendered well, and I proposed a couple minor updates in #8.

Regarding richfitz/storr#77 (comment), storr, and serialization in general (particularly if there is a way to avoid serialization while still being able to store and hash arbitrary data) I would love advice if you have the time. I think your expertise could really help speed up drake.

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