Skip to content

Conversation

@pablogsal
Copy link
Member

@pablogsal pablogsal commented Dec 13, 2025

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

Some thoughts:

  • To start, great idea :-)
  • Maybe put the images in a folder for organisation?
  • Did you optimise the pngs? (For one way to do this see, #139061)
  • What do you think about adding an example tachyon-flamegraph.html to _static?

@pablogsal
Copy link
Member Author

All done

@pablogsal pablogsal force-pushed the tachyon-images branch 3 times, most recently from dd0eb7b to d5a75fb Compare December 13, 2025 19:31
@hugovk hugovk mentioned this pull request Dec 13, 2025
@hugovk
Copy link
Member

hugovk commented Dec 13, 2025

To fix EPUB: pablogsal#116

@pablogsal
Copy link
Member Author

Thanks a lot @hugovk

pablogsal and others added 2 commits December 14, 2025 15:33
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

LGTM

@pablogsal pablogsal merged commit e4d3c83 into python:main Dec 14, 2025
46 checks passed
@pablogsal pablogsal deleted the tachyon-images branch December 14, 2025 17:43
@pablogsal
Copy link
Member Author

Thanks for the review everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants