pymc-devs / pymc-data-umbrella

Website: Data Umbrella & PyMC open source sessions
https://pymc-data-umbrella.xyz/en/latest/
MIT License
26 stars 38 forks source link

Update links in YouTube video descriptions #151

Closed zbraiterman closed 1 year ago

zbraiterman commented 1 year ago

How can community members best help update YouTube video descriptions for PyMC related videos on the Data Umbrella channel?

For example, how the link (https://github.com/pymc-devs/pymc-data-umbrella/blob/main/webinars/probabilistic_programming_with_pymc/notebook.ipynb) to the Jupyter Notebook for Austin’s Intro to Probabilistic Programming with PyMC video (https://www.youtube.com/watch?v=Qu6-_AnRCs8&t=1044s) is inactive.

@reshamas, is there a structure for these types of pull requests, like the one you’ve outlined here (https://github.com/pymc-devs/video-timestamps/issues/11)?

reshamas commented 1 year ago

Hi @zbraiterman, thanks for bringing this to our attention.

Link not working: https://github.com/pymc-devs/pymc-data-umbrella/blob/main/webinars/probabilistic_programming_with_pymc/notebook.ipynb

Maybe it's this: (a) https://github.com/pymc-devs/pymc-data-umbrella/blob/main/about/probabilistic_programming_with_pymc/notebook.ipynb

Maybe it's this: (b) https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

@OriolAbril What's the better link to refer to?

reshamas commented 1 year ago

@zbraiterman If you see more broken links like this, it's fine to open up an issue here, as you just did.

We restructured the whole sprint website, so that's why the links are broken, unfortunately.

reshamas commented 1 year ago

@OriolAbril @symeneses The "Run Code" button this page doesn't work. I am not sure how to fix that?

https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

OriolAbril commented 1 year ago

Maybe it's this: (b) https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

This one, always share links to the website.

We restructured the whole sprint website, so that's why the links are broken, unfortunately.

The links on the other webinars are not broken and never were. When restructuring I set up redirects so that the old links continue to work. However, as this link points to GitHub directly so setting up redirect was not possible.

I also remember the link being fixed already after flagging this on slack (link will only work if you have access to slack), could the change have been undone when adding the timestamps?

The "Run Code" button this page doesn't work. I am not sure how to fix that?

It is working for me, can you share a screenshot of what is happening in your case to try and see what might be going on?

reshamas commented 1 year ago

from slack:

Link to webinar page in website: https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/index.html (this I can and will make sure it never breaks even if we move the notebook)

It looks like there were two separate links in the video description, one directly to the repo and one to the html file. I do recall fixing one of them.

Confirming that I checked all the links in Austin's video description and they all work:

Key Links

Resources

reshamas commented 1 year ago

@OriolAbril Running notebook in the browser: to discuss later, next time we meet.

zbraiterman commented 1 year ago

Thank you, @reshamas and @OriolAbril, for following up on this.

Also, is the Slack open to / useful for users like myself?

reshamas commented 1 year ago

Also, is the Slack open to / useful for users like myself?

@zbraiterman Here is more information on the PyMC slack, including the "recurring contributor" role: https://github.com/pymc-devs/pymc/blob/main/GOVERNANCE.md#recurring-contributors

zbraiterman commented 1 year ago

Thank you, @reshamas, for sharing this information.

reshamas commented 1 year ago

@zbraiterman Thanks for opening this issue. I am closing this. If you see any more broken links, please open up a new issue here and I'll update the links.