LZDoom [beta 4 11/29 released]

Game Engines go in this forum

Re: LZDoom [beta 2 aka Helloween edition released]

Postby Rachael » Sat Dec 01, 2018 5:22 pm

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
Rachael
Webmaster
 
Joined: 13 Jan 2004
Discord: Rachael#3767
Twitch ID: madamerachelle
Github ID: madame-rachelle

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Sun Dec 02, 2018 8:15 am

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
drfrag
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Postby Rachael » Sun Dec 02, 2018 8:27 am

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
Rachael
Webmaster
 
Joined: 13 Jan 2004
Discord: Rachael#3767
Twitch ID: madamerachelle
Github ID: madame-rachelle

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Sun Dec 02, 2018 2:59 pm

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
drfrag
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Postby Rachael » Sun Dec 02, 2018 9:16 pm

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
Rachael
Webmaster
 
Joined: 13 Jan 2004
Discord: Rachael#3767
Twitch ID: madamerachelle
Github ID: madame-rachelle

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Mon Dec 03, 2018 7:40 am

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
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Mon Dec 03, 2018 11:41 am

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 allExpand view
$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
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Mon Dec 03, 2018 3:07 pm

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
drfrag
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 2 aka Helloween edition released]

Postby Rachael » Mon Dec 03, 2018 4:59 pm

Cool.

Thank you. :)
User avatar
Rachael
Webmaster
 
Joined: 13 Jan 2004
Discord: Rachael#3767
Twitch ID: madamerachelle
Github ID: madame-rachelle

Re: LZDoom [beta 2 aka Helloween edition released]

Postby drfrag » Mon Dec 03, 2018 6:26 pm

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
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 3 11/29 released]

Postby drfrag » Wed Dec 05, 2018 7:28 am

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 9:09 am, edited 1 time in total.
User avatar
drfrag
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 3 11/29 released]

Postby dpJudas » Wed Dec 05, 2018 7:34 am

Are you talking about a crash with sky caps in GZDoom itself, or in the attempt to backport it?
dpJudas
 
 
 
Joined: 28 May 2016

Re: LZDoom [beta 3 11/29 released]

Postby drfrag » Wed Dec 05, 2018 9:13 am

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
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 4 11/29 released]

Postby drfrag » Tue Dec 11, 2018 1:55 pm

I released beta 4 yesterday just to fix the glowing flats bug, GL 2 cards were not affected.
User avatar
drfrag
I.R developer, I.R smart
Vintage GZDoom Developer
 
Joined: 23 Apr 2004
Location: Spain

Re: LZDoom [beta 4 11/29 released]

Postby kondoriyano » Mon Dec 17, 2018 7:46 pm

i found some issue when playing with this music jukebox
viewtopic.php?f=46&t=49073
its like the music loop is interrupted
User avatar
kondoriyano
 
Joined: 03 Jul 2015
Location: AALIENS.WAD Map24

PreviousNext

Return to Game Engines

Who is online

Users browsing this forum: No registered users and 0 guests