Skip to content

Convert most bio pics to jpg to save network bandwidth (i.e., load time) #810

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

Merged
merged 2 commits into from
Aug 8, 2025

Conversation

jasongrout
Copy link
Member

@jasongrout jasongrout commented Aug 8, 2025

We retain a few bio pics as png that are line art, not amenable to jpg compression.

@jasongrout jasongrout changed the title Convert most bio pics to jpg to save space Convert most bio pics to jpg to save network bandwidth and space Aug 8, 2025
@jasongrout jasongrout requested a review from choldgraf August 8, 2025 02:07
@jasongrout jasongrout changed the title Convert most bio pics to jpg to save network bandwidth and space Convert most bio pics to jpg to save network bandwidth (i.e., load time) Aug 8, 2025
@jasongrout
Copy link
Member Author

Link check fixed at #812

We retain a few png bio pics that are line art, not amenable to jpg compression
@jasongrout
Copy link
Member Author

rebased on master so the about page works

@jasongrout jasongrout removed the request for review from choldgraf August 8, 2025 02:30
@jasongrout jasongrout requested a review from choldgraf August 8, 2025 02:33
@jasongrout
Copy link
Member Author

The about page works in the preview. For example, here is the EC, which had Chris and Rick's photos converted from png to jpg:

image

@choldgraf
Copy link
Collaborator

sweet - it looks good to me. In a future iteration we might also want to run oxipng on the images in our site in a one-time pass. will merge this for now though.

@rpwagner your image is badass

@choldgraf choldgraf merged commit e516e7e into jupyter:main Aug 8, 2025
4 of 5 checks passed
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