Skip to content

Conversation

@mmcky
Copy link
Contributor

@mmcky mmcky commented Feb 5, 2023

No description provided.

@netlify
Copy link

netlify bot commented Feb 5, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit e5f0cfc
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/63e0e63cbfd93a0008d6216a
😎 Deploy Preview https://deploy-preview-51--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.

@mmcky mmcky added the lecture label Feb 5, 2023
@shlff
Copy link
Member

shlff commented Feb 6, 2023

Hi @mmcky I've reviewed this lecture again and found some typos.

Should I propose those changes here or create another PR?

@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

Thanks @shlff this is the place to add them. 👍

@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

@shlff or @Smit-create there seems to be an error in the gated directives being reported by sphinx-exercise

ERROR: The document (markov_chains) is missing a solution-start directive

  solution-end at line: 1397
  solution-start at line: 1441
  solution-end at line: 1481
  solution-start at line: 1519
  solution-end at line: 1547

Could you check that as well

@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

@HumphreyYang this is building now -- so should give you a preview once you commit the updates.

@shlff
Copy link
Member

shlff commented Feb 6, 2023

Hi @HumphreyYang I pushed a commit with possible typos. Feel free to edit it.

@HumphreyYang
Copy link
Member

HumphreyYang commented Feb 6, 2023

Hi @mmcky,

The action failed with the error:

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Error: Process completed with exit code 100.

From the history, it seems to have difficulty downloading the docker from azure.archive.ubuntu.com:http: [IP: 20.106.104.242 80]?

@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

yeah looks like the azure cache is timing out. I will disable the texlive install and just focus on html for now

@github-actions github-actions bot temporarily deployed to commit February 6, 2023 10:18 Inactive
@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

@HumphreyYang looks like execution is failing

ERROR: Execution Failed: /home/runner/work/lecture-python-intro/lecture-python-intro/lectures/markov_chains.md

you may want to run this locally and see what the issue is directly.

@HumphreyYang
Copy link
Member

HumphreyYang commented Feb 6, 2023

@HumphreyYang looks like execution is failing

ERROR: Execution Failed: /home/runner/work/lecture-python-intro/lecture-python-intro/lectures/markov_chains.md

you may want to run this locally and see what the issue is directly.

Hi Matt,

I have run this locally before, and it runs without an error.

@mmcky
Copy link
Contributor Author

mmcky commented Feb 6, 2023

looks like it needs graphviz to convert some images

@HumphreyYang
Copy link
Member

HumphreyYang commented Feb 6, 2023

Many thanks! It is used to visualize the Markov chain.

@github-actions github-actions bot temporarily deployed to commit February 6, 2023 10:34 Inactive
@github-actions github-actions bot temporarily deployed to commit February 6, 2023 10:38 Inactive
@github-actions github-actions bot temporarily deployed to commit February 6, 2023 10:49 Inactive
@mmcky mmcky merged commit ead1bca into main Feb 6, 2023
@mmcky mmcky deleted the markov-chain branch February 6, 2023 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants