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

Add the Pong example to the index page #3

Conversation

jessica-mitchell
Copy link

This PR adds the example files into the index grid, along with modified image to scale properly. The gif was moved from pynest/examples to the docs/htmldoc/static/img folder. Do not want to duplicate as it's fairly large but it needs to be accessible for the docs. Also scaled the sudoku gif so it renders better.

See example output here: https://nest-test.readthedocs.io/en/add-pong-index/examples/index.html#pynest-examples

Copy link
Owner

@krachbumm3nte krachbumm3nte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great on the website. Regarding the rendering of the GIF files: Since I will have to change the backend of the Pong example from PIL to matplotlib aswell, do I have to add padding to the images too, or is this rather about the final image size of 400x300px?

@jessica-mitchell
Copy link
Author

do I have to add padding to the images too, or is this rather about the final image size of 400x300px?

The gifs have padding so not be stretched awkwardly in the card, but for for adapting to matplotlib, I dont think you have to worry about changing the output in any way.

@krachbumm3nte krachbumm3nte merged commit 65911a3 into krachbumm3nte:pong_pynest_example Sep 12, 2022
krachbumm3nte pushed a commit that referenced this pull request Oct 26, 2022
krachbumm3nte pushed a commit that referenced this pull request Sep 22, 2023
@jessica-mitchell jessica-mitchell deleted the add-pong-index branch April 17, 2024 18:10
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.

2 participants