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

thumbnails not showing on notebooks home page #479

Closed
reshamas opened this issue Dec 19, 2022 · 8 comments
Closed

thumbnails not showing on notebooks home page #479

reshamas opened this issue Dec 19, 2022 · 8 comments

Comments

@reshamas
Copy link
Contributor

Notebook title:
Notebook url:

Thumbnails are not showing here:
https://www.pymc.io/projects/examples/en/latest/gallery.html

Screen Shot 2022-12-19 at 12 29 42 PM

@twiecki
Copy link
Member

twiecki commented Dec 19, 2022

Probably due to pymc-devs/brand#21

reshamas added a commit to reshamas/pymc-examples that referenced this issue Dec 19, 2022
@reshamas
Copy link
Contributor Author

Arrgh! Disliking breaking changes. :(

@drbenvincent
Copy link
Contributor

These notebooks exist here https://github.com/pymc-devs/pymc/tree/main/docs/source/learn/core_notebooks rather than in the pymc-examples repo. Personally I think this creates a bit of added complexity. What do people think about moving them into the pymc-examples repo?

I think the logic of having them in pymc-devs/pymc was to make sure that these are always up to date with the latest API. But I think we can just achieve the same result if we treat these as 'core' and of high importance, but in the pymc-examples repo.

What do people think?

@twiecki
Copy link
Member

twiecki commented Dec 28, 2022 via email

@OriolAbril
Copy link
Member

trying to think about the workflow to ensure the core notebooks are always up to date with the codebase I find it more complicated. I am not sure users know or even care the page they go to when they click on those first cards are generated from a different repo.

How do you envision the updated workflow to be?

For comparison, he current one is any time a change breaks a core notebook, CI fails for that PR and the notebook is updated in the same PR that modified the related code. Every PR and every build on latest execute all core notebooks on the corresponding local github version.

@ricardoV94
Copy link
Member

I think it makes sense for the core notebooks to work as they do now. It must be possible to fix the thumbnails some other way.

@OriolAbril
Copy link
Member

Thumbnails are already fixed by the way, we just forgot to close the issue

@reshamas
Copy link
Contributor Author

Closed by #480

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

No branches or pull requests

5 participants