Skip to content

Conversation

@MaybeMaru
Copy link
Contributor

@MaybeMaru MaybeMaru commented Sep 27, 2025

Description

Saw this PR a while ago that had plans to be merged but was ultimately abandoned.
Here im doing the same thing as the old one, running all Vorbis files through OptiVorbis to optimize a bit of filesize with lossless quality. Which saves around 30mb of file size, not bad considering all original quality is preserved.
Hoping that now with the new Github team it'll be much smoother trying to merge a sizeable PR like this.

As a little note, aside from this PR, I've also noticed that the current vorbis audio files fluctuate a lot in quality. Some songs can take up to 4mb of size, while their erect counterparts only take 1mb. Could be worth something to look into.

@github-actions github-actions bot added status: pending triage Awaiting review. size: tiny A tiny pull request with 4 or fewer changes. pr: audio PR modifies sound or music. and removed status: pending triage Awaiting review. size: tiny A tiny pull request with 4 or fewer changes. labels Sep 27, 2025
@Hundrec
Copy link
Member

Hundrec commented Sep 27, 2025

Thanks for making this! I think before we can accept one of these, we need to fix a bunch of issues with a lot of the audio tracks.
So we'll have to wait a while on this until everything is taken care of
However, it seems to be an easy process, so I'll keep this open as a reminder that we should do this eventually.

@Hundrec Hundrec added status: pending triage Awaiting review. size: large A large pull request with more than 100 changes. type: optimization Involves a performance issue or a bug which causes lag. labels Sep 27, 2025
@MaybeMaru
Copy link
Contributor Author

MaybeMaru commented Sep 27, 2025

Alright, sounds good!
Let me know when all of those audio issues are handled so I can do an update of this PR

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

Labels

pr: audio PR modifies sound or music. size: large A large pull request with more than 100 changes. status: pending triage Awaiting review. type: optimization Involves a performance issue or a bug which causes lag.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants