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

data: switch from WEBP to JPEG #1469

Closed
wants to merge 1 commit into from
Closed

data: switch from WEBP to JPEG #1469

wants to merge 1 commit into from

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Aug 30, 2024

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

The files appear identical visually, but the title menu loads much faster with this format. On my machine it goes down from 236 ms to around 150 ms. However, this bloats the asset size by 34% (9.4 MiB compared to 7.0 MiB).

@rr- rr- added the Enhancement Improvement of an existing feature label Aug 30, 2024
@rr- rr- added this to the 4.4 milestone Aug 30, 2024
@rr- rr- self-assigned this Aug 30, 2024
@rr- rr- requested review from a team as code owners August 30, 2024 21:20
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team August 30, 2024 21:20
Copy link
Collaborator

@walkawayy walkawayy left a comment

Choose a reason for hiding this comment

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

Looks good except you missed the title, credits, and Eidos images in the UB gameflow file.

@rr- rr- force-pushed the webp-to-jpeg branch 2 times, most recently from 2083dfd to b32592a Compare August 30, 2024 21:30
@rr- rr- requested a review from walkawayy August 30, 2024 21:30
Copy link
Collaborator

@aredfan aredfan left a comment

Choose a reason for hiding this comment

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

LGTM.

@rr-
Copy link
Collaborator Author

rr- commented Aug 31, 2024

With the recent improvements on develop and #1470 the savings of this pull request went down to 15 milliseconds (130 ms vs 115 ms). I'll hold on for a bit longer to see if we can do better on other fronts prior to making this switch.

@rr-
Copy link
Collaborator Author

rr- commented Sep 1, 2024

All in all I don't think it's worth it anymore as the savings aren't so dramatic with the recent improvements on develop – especially considering the file size increase. Closing for now.

@rr- rr- closed this Sep 1, 2024
@rr- rr- deleted the webp-to-jpeg branch September 1, 2024 19:01
@rr- rr- added the TR1 label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement of an existing feature TR1
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants