Is there any way that I can help you? Do you remember the conflicts you had with that merge?drfrag wrote:It's a bad merge of "Add "neutral" gender option and better obit formatting " somehow but there was no conflict. The 3.51 beta was not affected.
I'll investigate it. Setting the gender to object works, strange.
LZDoom 3.88b 02/26 released
Forum rules
The Projects forums are ONLY for YOUR PROJECTS! If you are asking questions about a project, either find that project's thread, or start a thread in the General section instead.
Got a cool project idea but nothing else? Put it in the project ideas thread instead!
Projects for any Doom-based engine are perfectly acceptable here too.
Please read the full rules for more details.
The Projects forums are ONLY for YOUR PROJECTS! If you are asking questions about a project, either find that project's thread, or start a thread in the General section instead.
Got a cool project idea but nothing else? Put it in the project ideas thread instead!
Projects for any Doom-based engine are perfectly acceptable here too.
Please read the full rules for more details.
-
- Posts: 13793
- Joined: Tue Jan 13, 2004 1:31 pm
- Preferred Pronouns: She/Her
Re: LZDoom [beta 2 aka Helloween edition released]
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
Thanks, if you could help i'd really appreciate it. Technically it's not a bad merge, there weren't any conflicts. The code is pretty much the same. Following s_advsound.cpp history the only difference is that "Removed all superfluous #include's" is not there.
This is not easy to debug, i've only managed to see that listidx = 65535 when gender is not OBJECT (else it's 3) and in static int S_LookupPlayerSound the engine enters in if (listidx == 0xffff) and if (g == GENDER_MAX).
I've noticed that there are three versions of S_LookupPlayerSound, i'm referring to the one defined in line 1768.
This is not easy to debug, i've only managed to see that listidx = 65535 when gender is not OBJECT (else it's 3) and in static int S_LookupPlayerSound the engine enters in if (listidx == 0xffff) and if (g == GENDER_MAX).
I've noticed that there are three versions of S_LookupPlayerSound, i'm referring to the one defined in line 1768.
-
- Posts: 13793
- Joined: Tue Jan 13, 2004 1:31 pm
- Preferred Pronouns: She/Her
Re: LZDoom [beta 2 aka Helloween edition released]
Unfortunately... I know next to nothing about the sound system. When I offered to help I thought you were referring to that specific commit - something I did know a bit about.
Is listidx the same in both source ports? Have you debugged GZDoom itself with this issue? I will say, it sounds almost as if the references between "male" and "female" were completely broken in that commit. But why it reverts to Doomguy screams and grunts, is pretty strange.
Is listidx the same in both source ports? Have you debugged GZDoom itself with this issue? I will say, it sounds almost as if the references between "male" and "female" were completely broken in that commit. But why it reverts to Doomguy screams and grunts, is pretty strange.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
I've checked the files in that commit with difmerge at that point and the only difference between LZDoom and the vintage build is a few includes, not that all of them are really needed. The changes to s_advsound.cpp look very innocent (and to all those files) and i'm almost sure the problem is not in the sound system at all. Must be something else but no idea of what happens. The references to listidx are the same. It's essentially the same code.
-
- Posts: 13793
- Joined: Tue Jan 13, 2004 1:31 pm
- Preferred Pronouns: She/Her
Re: LZDoom [beta 2 aka Helloween edition released]
I've tried to look at this but nothing I see is indicating to me that there should be any issue. Whatever the issue is, it's buried pretty deep.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
Thanks again for looking into it. That or it's a pretty silly thing somehow related to the includes/different layout.
Looks like a brute force code analysis is in order (DiffMerge+Blame), shouldn't take too much.
Looks like a brute force code analysis is in order (DiffMerge+Blame), shouldn't take too much.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
I've noticed that Heresy II (tested with Doom II, it's for Heretic) works well selecting the baron player.
Why does junoplayer appear twice? That's the player class twice and not the gender. Is gender optional? Is this a bad definition or a custom gender?
https://zdoom.org/wiki/SNDINFO
Edit: i see that if gender is not identified the default is object now and not male (d_netinfo.cpp), that could be the problem. But i don't know why it works in GZDoom.
Why does junoplayer appear twice? That's the player class twice and not the gender. Is gender optional? Is this a bad definition or a custom gender?
https://zdoom.org/wiki/SNDINFO
Code: Select all
$playersound baron male *death dsbrsdth
$playersound junoplayer junoplayer *death death
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
I've pushed a fix here:
https://github.com/drfrag666/gzdoom/com ... 6bb82afd83
I've managed to reproduce the issue with the vintage build (recent devbuild), remember that 3.60b2 is ahead 3.6.0.
https://github.com/drfrag666/gzdoom/com ... 6bb82afd83
I've managed to reproduce the issue with the vintage build (recent devbuild), remember that 3.60b2 is ahead 3.6.0.
-
- Posts: 13793
- Joined: Tue Jan 13, 2004 1:31 pm
- Preferred Pronouns: She/Her
Re: LZDoom [beta 2 aka Helloween edition released]
Cool.
Thank you.
Thank you.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 2 aka Helloween edition released]
You're welcome. I made the same mistake and compared with 3.6.0_legacy, also i should have checked the mod better.
You can add me as remote and cherry-pick the fix if you want, It's not in the legacy branch yet since i'm merging a lot of stuff from the asmjit branch and i haven't compiled yet (it would take a lot of time). I couldn't merge directly or else i would get a fake history.
You can add me as remote and cherry-pick the fix if you want, It's not in the legacy branch yet since i'm merging a lot of stuff from the asmjit branch and i haven't compiled yet (it would take a lot of time). I couldn't merge directly or else i would get a fake history.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 3 11/29 released]
I've released a new beta with the fix (see first post). While technically it's not a bug (it's a mod side one) and it's a minor thing i've pushed a fix since there must be a fallback default for the sound class gender when it's not properly defined and that must be male not other since that's the default value for the gender cvar (doomguy it's not an object). So please tell Impie it's a problem with the mod itself.
Of course the dither shader had to go so no more intel fixed versions (not needed anymore).
This delayed beta is in sync (is that term even correct?) with GZ master as of 11/29 to clarify things.
I could not port the recent SSE optimizations to the poly renderer (and the poly renderer is very important in LZDoom).
There was a crash in resolutions > 640 with "removed the 8x8 block drawing code from softpoly" related to sky caps, of course i could try to fix that. But then "change DrawSpanOpt32 to render a scanline in multiple steps" causes a black screen problem with broken geometry in the menu, it only works after starting the software renderer first after engine launch. That's why it requires "change the software renderer to render to a DSimpleCanvas like the old D3D9 target did. Then use the drawer threads to memcpy the result to the PBO" which itself requires some things in "added a software scene drawer to the GL renderer".
Just in case someone is interested, as usual any hints/help would be appreciated.
Also i've pushed a lot of stuff for the legacy build, looks like Graf is working on the engine full time. Does he even sleep? So i'm getting close but still not there.
Of course the dither shader had to go so no more intel fixed versions (not needed anymore).
This delayed beta is in sync (is that term even correct?) with GZ master as of 11/29 to clarify things.
I could not port the recent SSE optimizations to the poly renderer (and the poly renderer is very important in LZDoom).
There was a crash in resolutions > 640 with "removed the 8x8 block drawing code from softpoly" related to sky caps, of course i could try to fix that. But then "change DrawSpanOpt32 to render a scanline in multiple steps" causes a black screen problem with broken geometry in the menu, it only works after starting the software renderer first after engine launch. That's why it requires "change the software renderer to render to a DSimpleCanvas like the old D3D9 target did. Then use the drawer threads to memcpy the result to the PBO" which itself requires some things in "added a software scene drawer to the GL renderer".
Just in case someone is interested, as usual any hints/help would be appreciated.
Also i've pushed a lot of stuff for the legacy build, looks like Graf is working on the engine full time. Does he even sleep? So i'm getting close but still not there.
Last edited by drfrag on Wed Dec 05, 2018 8:09 am, edited 1 time in total.
-
-
- Posts: 3134
- Joined: Sat May 28, 2016 1:01 pm
Re: LZDoom [beta 3 11/29 released]
Are you talking about a crash with sky caps in GZDoom itself, or in the attempt to backport it?
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 3 11/29 released]
Sorry i meant > 640 and it happens only with the old D3D and OGL backends, the vintage build runs fine. I reverted that commit but the main problem is the black screen thing.
-
- Vintage GZDoom Developer
- Posts: 3150
- Joined: Fri Apr 23, 2004 3:51 am
- Location: Spain
Re: LZDoom [beta 4 11/29 released]
I released beta 4 yesterday just to fix the glowing flats bug, GL 2 cards were not affected.
-
- Posts: 13
- Joined: Fri Jul 03, 2015 5:38 pm
- Location: AALIENS.WAD Map24
Re: LZDoom [beta 4 11/29 released]
i found some issue when playing with this music jukebox
viewtopic.php?f=46&t=49073
its like the music loop is interrupted
viewtopic.php?f=46&t=49073
its like the music loop is interrupted