JACoders / OpenJK

Community effort to maintain and improve Jedi Academy (SP & MP) + Jedi Outcast (SP only) released by Raven Software
GNU General Public License v2.0
2.01k stars 612 forks source link

SP combat music slow to transition back to exploration music. #915

Closed JokerOmega1 closed 7 years ago

JokerOmega1 commented 7 years ago

Reporting a bug? Please make sure you've given the following information - thanks!

Operating system and version: Windows 10, openjk-2016-08-22-cdfdb5ad-windows

Is this for single player or multiplayer? Single player

Description of the bug (and if possible, steps to reproduce the bug): Combat music is very slow to transition back to exploration music after defeating enemies, some times not transitioning at all.

What did you expect to happen instead? Music to transition as it should.

ensiform commented 7 years ago

Are you able to build from source to see if it is still the same? The buildbot building releases as evidenced by 8-22 is moderately old now. Can you check what build it stopped working if any? The only thing I can think of is maybe @mrwonko's changes to DMS system to rewrite parsers maybe messed something up in transitioning without noticing it being broken.

JokerOmega1 commented 7 years ago

I am not exactly sure how to "build from source", is it a download option I am missing perhaps?

Razish commented 7 years ago

Just to confirm, are you saying the transition is slower than it was in vanilla JKA, making this a bug? Or is this a feature request?

JokerOmega1 commented 7 years ago

I am reporting this as a bug,

"Description of the bug (and if possible, steps to reproduce the bug): Combat music is very slow to transition back to exploration music after defeating enemies, some times not transitioning at all."

Razish commented 7 years ago

Okay thanks, just wanted to confirm before we go chasing down behavior changes we may not have made :P I think Ensi is on the right track regarding the parser rewrite FWIW

ensiform commented 7 years ago

I also noticed on a level in jk2 there is some sort of warning about useboss and no boss text defined. I forgot what level it was on though. Could be related to issues in parser too.

JokerOmega1 commented 7 years ago

Hope you can locate the cause to this issue, holding off on another JKJA SP playthrough until this has been fixed, hopefully you'll release another windows build when that happens.

Bucky21659 commented 7 years ago

I also noticed on a level in jk2 there is some sort of warning about useboss and no boss text defined. I forgot what level it was on though. Could be related to issues in parser too.

Seeing this when loading a savegame on artus_detention, console outputs: Error parsing music data (in "ext_data/dms.dat"): 'useboss' "1" has no "boss" entry!

No music started playing until I did vid_restart to copy the error from console, and it's just the exploration music.

ensiform commented 7 years ago

Paging @mrwonko ^

mrwonko commented 7 years ago

I'll take a look later.

JokerOmega1 commented 7 years ago

Yay progress!

mrwonko commented 7 years ago

There's a chance my commit b11d6dd caused this with its changes to snd_music.cpp, I'll take a closer look at the weekend when I'm not exhausted from 8h of work.

mrwonko commented 7 years ago

Huh, that was simple enough... Thanks for including the 'useboss' "1" has no "boss" entry! error, that helped a lot. Fix coming momentarily, and it was indeed my fault.

mrwonko commented 7 years ago

I'm not completely sure this also fixes the slow transition issue, please reopen if that is still an issue.

mrwonko commented 7 years ago

Here's a binary for you to test with if you're on Windows. openjk_sp.x86.exe.zip

JokerOmega1 commented 7 years ago

My game doesn't start when I attempt to use that binary, I have attached the crash log, if it helps. crashlog-2017-05-12_18-16-10.txt

mrwonko commented 7 years ago

Ugh... Looks like you have an outdated renderer. I'm not sure if there have been any binary releases of a compatible version. Maybe there are - otherwise I'll need to build that and probably jagamex86.dll once I get around to it. (Currently on my Mac...)

JokerOmega1 commented 7 years ago

Well I am using the latest windows release "bot" build (openjk-2016-08-22-cdfdb5ad-windows), so if you manage to put together up to date files for this than that would be swell, would love to get back to my playthrough without the annoying music issue.

ensiform commented 7 years ago

That version isn't compatible with later versions of the code since then. @mrwonko just was being lazy and only sent you the exe by itself without knowing the renderer and gamecode should have been included.

mrwonko commented 7 years ago

Right, sorry about that, this one should hopefully include everything you need: OpenJK-Windows-x86-JKASP.zip Potentially also of interest: OpenJK-Windows-x86-JKAMP.zip

JokerOmega1 commented 7 years ago

So far so good, the slow music transition seems to be gone, thanks for fix mrwonko!