Skip to content

Conversation

@mmcky
Copy link
Contributor

@mmcky mmcky commented Apr 17, 2023

This PR adds the graphviz python interface package install to the top of lake_model and eigen_II.md

  • @HengchengZhang could you also do a grep across all md files for graphviz and make sure all lectures that use it have it installed.
@netlify
Copy link

netlify bot commented Apr 17, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 23b81da
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/643ceee245d3ef0008667b57
😎 Deploy Preview https://deploy-preview-163--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot temporarily deployed to commit April 17, 2023 06:32 Inactive
@HengchengZhang
Copy link
Member

Hi @mmcky, I've checked the rest markdown files and both markov_chains_I.md and markov_chains_II.md used graphviz.

In both lectures graphviz was installed by pip but missing the documentation to graphviz

:class: warning If you are running this lecture locally it requires [graphviz](https://www.graphviz.org) to be installed on your computer. Installation instructions for graphviz can be found [here](https://www.graphviz.org/download/) 
@mmcky
Copy link
Contributor Author

mmcky commented Apr 17, 2023

thanks @HengchengZhang can you please update the PR when you are ready. That would be great.

@github-actions github-actions bot temporarily deployed to commit April 17, 2023 07:06 Inactive
@HengchengZhang
Copy link
Member

Updated @mmcky, many thanks.

@mmcky
Copy link
Contributor Author

mmcky commented Apr 18, 2023

@mmcky mmcky merged commit d1281e3 into main Apr 18, 2023
@mmcky mmcky deleted the fix-graphviz branch April 18, 2023 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants