Skip to content

Further reduce PNG size in README (using tinyimg)#572

Closed
zeileis wants to merge 2 commits into
mainfrom
tinyimg
Closed

Further reduce PNG size in README (using tinyimg)#572
zeileis wants to merge 2 commits into
mainfrom
tinyimg

Conversation

@zeileis

@zeileis zeileis commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

Grant @grantmcdermott, prompted by the discussion at https://bsky.app/profile/gmcd.bsky.social/post/3mkgqzndju22m I tweaked the README further so that (a) PNGs are now 960 x 600 pixels (rather than 1600 x 1000) and (b) post-processed by tinyimg (if available). This reduces the pkg size by another few hundred K.

Feel free to reject if you think that's too extreme...

@grantmcdermott

Copy link
Copy Markdown
Owner

Thanks @zeileis. I was waiting to regenerate these README images based the recent aesthetic changes that I've been implementing for v0.7.0. Would you mind merging the recent changes from main into your local branch and then re-generating these reduced images again, so I can check?

Apologies for the churn...

@zeileis

zeileis commented May 17, 2026

Copy link
Copy Markdown
Collaborator Author

No worries, happy to do so.

But maybe wait some more until all recent issues/PRs have been incorporated?

@grantmcdermott

Copy link
Copy Markdown
Owner

Good idea. I have one or two more big "aesthetic" PRs to finish and then hope to submit 0.7.0 after that. I'll ping you once that's ready.

grantmcdermott added a commit that referenced this pull request Jul 3, 2026
* example gotcha

* typos

* version bump

* cran comments

* exclude tests for CRAN

* co-opt pr #572

* we still need tne tests on the CI

* ci(revdep): add setup-r so collect job has Rscript

* docs(cran): note revdep results (RTMB install failure unrelated)

* submitted

* revert tests for CI
@grantmcdermott

Copy link
Copy Markdown
Owner

I bundled these into #658 :-)

@zeileis

zeileis commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator Author

OK, thanks! I was wondering about this 😅

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