LZDoom 3.88b 02/26 released

Game Engines like EDGE, LZDoom, QZDoom, ECWolf, and others, go in this forum
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.
User avatar
Rachael
Posts: 13723
Joined: Tue Jan 13, 2004 1:31 pm
Preferred Pronouns: She/Her

Re: LZDoom [beta 2 aka Helloween edition released]

Post by Rachael »

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.
Is there any way that I can help you? Do you remember the conflicts you had with that merge?
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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.
User avatar
Rachael
Posts: 13723
Joined: Tue Jan 13, 2004 1:31 pm
Preferred Pronouns: She/Her

Re: LZDoom [beta 2 aka Helloween edition released]

Post by Rachael »

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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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. :?
User avatar
Rachael
Posts: 13723
Joined: Tue Jan 13, 2004 1:31 pm
Preferred Pronouns: She/Her

Re: LZDoom [beta 2 aka Helloween edition released]

Post by Rachael »

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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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

Code: Select all

$playersound	baron	male	*death		dsbrsdth

$playersound           junoplayer	 junoplayer	 *death	    death
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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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.
User avatar
Rachael
Posts: 13723
Joined: Tue Jan 13, 2004 1:31 pm
Preferred Pronouns: She/Her

Re: LZDoom [beta 2 aka Helloween edition released]

Post by Rachael »

Cool.

Thank you. :)
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Post by drfrag »

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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 3 11/29 released]

Post by drfrag »

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.
Last edited by drfrag on Wed Dec 05, 2018 8:09 am, edited 1 time in total.
dpJudas
 
 
Posts: 3109
Joined: Sat May 28, 2016 1:01 pm

Re: LZDoom [beta 3 11/29 released]

Post by dpJudas »

Are you talking about a crash with sky caps in GZDoom itself, or in the attempt to backport it?
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 3 11/29 released]

Post by drfrag »

Sorry i meant > 640 :oops: 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.
User avatar
drfrag
Vintage GZDoom Developer
Posts: 3146
Joined: Fri Apr 23, 2004 3:51 am
Location: Spain

Re: LZDoom [beta 4 11/29 released]

Post by drfrag »

I released beta 4 yesterday just to fix the glowing flats bug, GL 2 cards were not affected.
User avatar
kondoriyano
Posts: 13
Joined: Fri Jul 03, 2015 5:38 pm
Location: AALIENS.WAD Map24

Re: LZDoom [beta 4 11/29 released]

Post by kondoriyano »

i found some issue when playing with this music jukebox
viewtopic.php?f=46&t=49073
its like the music loop is interrupted

Return to “Game Engines”