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

Music: Add new relative_speed option in the config layer #672

Conversation

myst6re
Copy link
Contributor

@myst6re myst6re commented Mar 3, 2024

Summary

External Music: ability to set music speed via config

Motivation

Kurodo asked me if it was possible to sync "tb" music track (main theme alternative / Cloud's mind) with the scene, like in PS1
https://www.youtube.com/watch?v=ELt0-heDj0I&t=20020s

The PC version is faster than the PSX version, one solution would be speed up the music. That's increase the pitch a little, I'm not a big fan of this solution. But everyone seems fine with it, for example the modder Postscriptthree released a fix that just does just that, DLPB also did this in the past.

Since this patch is really light, and can be used by modders for whetever creative reasons (I enjoy listening to the slow version of costa del sol, for example), so let's do it!

ACKs

  • I have updated the Changelog.md file
  • I did test my code on FF7
  • I did test my code on FF8

@myst6re myst6re force-pushed the feature/config-change-music-speed branch from abd6139 to 17bf1bc Compare March 3, 2024 20:37
@myst6re myst6re marked this pull request as ready for review March 3, 2024 20:47
@myst6re myst6re force-pushed the feature/config-change-music-speed branch from 17bf1bc to 3296b5b Compare March 3, 2024 20:47
@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Mar 3, 2024
@julianxhokaxhiu julianxhokaxhiu added this to the 1.19.0 milestone Mar 3, 2024
julianxhokaxhiu
julianxhokaxhiu previously approved these changes Mar 3, 2024
Changelog.md Outdated Show resolved Hide resolved
@julianxhokaxhiu julianxhokaxhiu changed the title External Music: ability to set music speed via config Music: Add new relative_speed option in the config layer Mar 3, 2024
@julianxhokaxhiu julianxhokaxhiu merged commit 1e8b467 into julianxhokaxhiu:master Mar 3, 2024
@julianxhokaxhiu
Copy link
Owner

Thanks!

@myst6re myst6re deleted the feature/config-change-music-speed branch March 14, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants