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

Adventure map, sound of the horse's hooves continues after the horse has reached its destination #8375

Open
2 tasks done
LeHerosInconnu opened this issue Feb 8, 2024 · 4 comments
Labels
audio Sound and music related stuff bug Something doesn't work

Comments

@LeHerosInconnu
Copy link

Preliminary checks

Platform

Windows

Describe the bug

Windows 7 SP1 - 64 bits

In fheroes2, the sound of the horse's hooves continues after the horse has reached its destination.

In fheroes2:
Activate the sound of the video.

2024-02-08.12-22-32.mp4

Save file

Here is the save file:
Horse sound after move 01.zip

Additional info

Windows 7 SP1 - 64 bits

@LeHerosInconnu LeHerosInconnu added the bug Something doesn't work label Feb 8, 2024
@oleg-derevenetz oleg-derevenetz added audio Sound and music related stuff low priority Low priority item which could be addressed in the future labels Feb 8, 2024
@oleg-derevenetz oleg-derevenetz added this to the Beyond OG scope milestone Feb 8, 2024
@oleg-derevenetz oleg-derevenetz removed the low priority Low priority item which could be addressed in the future label Feb 8, 2024
@Branikolog
Copy link
Collaborator

Hi, @LeHerosInconnu
Are you using MIDI? I believe this issue is valid only for MIDI sound, while for external music I cannot reproduce this problem.

@LeHerosInconnu
Copy link
Author

Hello @Branikolog,

Hi, @LeHerosInconnu Are you using MIDI? I believe this issue is valid only for MIDI sound, while for external music I cannot reproduce this problem.

Yes, MIDI is used when the problem is detected.
I did a test with .ogg files in the MUSIC folder and "Music Type" set to "External" and the problem is no longer present.
There must have been a change in MIDI management at some point.

@oleg-derevenetz
Copy link
Collaborator

There must have been a change in MIDI management at some point.

This was always the case. MIDI playback (more precisely, MIDI playback startup) is slow in SDL_mixer. Here is the video of fheroes2 1.0.0 released Dec 21 2022 (more than a year ago):

fheroes2.engine.version_.1.0.0.2024-02-10.20-57-11.mp4

As you can see, there is the same pause that caused by delay of playing the WAV musical effect for this Shrine caused by MIDI soundtrack change due to the change of terrain. This is the same place, but using the Windows built-in MIDI (not FluidSynth MIDI) - i removed the soundfont file to force SDL_mixer to switch to the Windows MIDI:

fheroes2.engine.version_.1.0.0.2024-02-10.20-58-03.mp4

The same delay (approximately +/-). We tried to mitigate this at the time by moving the background music playback to a separate thread, and it's just that when crossing the border of the terrain it's almost unnoticeable (except for some extra horse step sounds maybe), but in situations like this it will be noticeable with MIDI, because SDL_mixer cannot start MIDI music (which, again, is a rather slow operation) and play WAV file at the same time, because it's not thread-safe. There's not much we can do about it right now. Do not use MIDI :)

@oleg-derevenetz
Copy link
Collaborator

oleg-derevenetz commented Feb 10, 2024

This delay is not observed with prehistoric 0.9.11, but the only reason is that in that version background music changes AFTER the object is visited, consider (turn the video sound on):

Free.Heroes.of.Might.and.Magic.II.version_.0.9.11.2024-02-10.21-19-18.mp4

And the MIDI playback itself was problematic even then, it was just a bit disguised in this particular case. Regarding horse steps, in this video (also with prehistoric 0.9.11) you can hear how the horse steps sounds continue a little even after the hero stops on the new terrain, because their playback has been delayed a bit during the MIDI music change due to change of terrain:

Free.Heroes.of.Might.and.Magic.II.version_.0.9.11.2024-02-10.21-29-49.mp4

And so far it can't be any other way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audio Sound and music related stuff bug Something doesn't work
Projects
None yet
Development

No branches or pull requests

3 participants