Relighting v4.0152b - smarter lighting

[Hey Doomer_]

Forgot to report but I received a vm abort with last three versions on drpg + drla

VM execution aborted: array access out of bounds. Negative current index = -1

Called from hd_relighting_EventHandler.WorldLoaded at relighting v4.0144b.pk3:zscript/hd_relighting.zs, line 810
]

This looks like the sector areas code.

Code: Select all

		for(int i = 0; i < secs.Size(); i++)
		{
			if (secs[i].area >= rl_Cvar.rl_sector_area_min.GetInt() && secs[i].area <= max_area) secareas.Push(secs[i].area);
		}

secareas is populated if the sector areas are between the min and max areas... Curious if this happens if you lower the min sec area... otherwise this is a possible workaround for line 810:

Code: Select all

		double min_area = rl_Cvar.rl_sector_area_min.GetInt();
		if (secareas.Size()) min_area = skewness > 1.0 ? clamp(secareas[secareas.Size() - 1] * skewness, rl_Cvar.rl_sector_area_min.GetInt(), secareas[0] * 0.25) : secareas[secareas.Size() - 1];

This should work since all the skew does is correct for maps with many small sectors.

[thugsta]

Out of curiosity, what happened to having a compatible version for LZ that i sent you?

On a side note, how do i go about fixing an equivalent for LZ to the errors Unknown identifier 'PPShader' and the error Unknown identifier 'Lightlevel'

Thanks, keep up the fantastic work as this is great.

[cosmos10040]

Forgot to report but I received a vm abort with last three versions on drpg + drla

VM execution aborted: array access out of bounds. Negative current index = -1

Called from hd_relighting_EventHandler.WorldLoaded at relighting v4.0144b.pk3:zscript/hd_relighting.zs, line 810
]

This looks like the sector areas code.

Code: Select all

		for(int i = 0; i < secs.Size(); i++)
		{
			if (secs[i].area >= rl_Cvar.rl_sector_area_min.GetInt() && secs[i].area <= max_area) secareas.Push(secs[i].area);
		}

secareas is populated if the sector areas are between the min and max areas... Curious if this happens if you lower the min sec area... otherwise this is a possible workaround for line 810:

Code: Select all

		double min_area = rl_Cvar.rl_sector_area_min.GetInt();
		if (secareas.Size()) min_area = skewness > 1.0 ? clamp(secareas[secareas.Size() - 1] * skewness, rl_Cvar.rl_sector_area_min.GetInt(), secareas[0] * 0.25) : secareas[secareas.Size() - 1];

This should work since all the skew does is correct for maps with many small sectors.

Looks like that worked! Thanks!

[Hey Doomer_]

Out of curiosity, what happened to having a compatible version for LZ that i sent you?

On a side note, how do i go about fixing an equivalent for LZ to the errors Unknown identifier 'PPShader' and the error Unknown identifier 'Lightlevel'

Thanks, keep up the fantastic work as this is great.

Holy smokes! I had forgotten about LZ, thanks for reminding me. I've been preoccupied with baked lighting issues.

PPShader is for GZDoom shaders any may just be incompatible. It sets parameters for the shader and would have to be commented out, I would guess.

I'd have to know where you're getting the "Lightlevel" error; this could be related to actors, and that map be something else that is just incompatible with LZ.

[hitmanx]

For some reason the combination of this mod and the mildoom 3 mod cause the framerate to steady plummet all the way down to single digits on map02 of doom 2.

No idea why, just thought you might like to know. Loving the mod otherwise

[thugsta]

Out of curiosity, what happened to having a compatible version for LZ that i sent you?

On a side note, how do i go about fixing an equivalent for LZ to the errors Unknown identifier 'PPShader' and the error Unknown identifier 'Lightlevel'

Thanks, keep up the fantastic work as this is great.

Holy smokes! I had forgotten about LZ, thanks for reminding me. I've been preoccupied with baked lighting issues.

PPShader is for GZDoom shaders any may just be incompatible. It sets parameters for the shader and would have to be commented out, I would guess.

I'd have to know where you're getting the "Lightlevel" error; this could be related to actors, and that map be something else that is just incompatible with LZ.

Yes i had it commented out but was hoping for a better solution that is "good enough" unfortunately

The Lightlevel error is at around 496 if i remember correct in the hd_actors.zs at the part where it has the caster.Lightlevel = int( etc. etc.

[Dan_The_Noob]

map12 is a little bit dark outside. infact it's pitch black almost. considering it's outside, maybe need a sky check or something?

[kalensar]

map12 is a little bit dark outside. infact it's pitch black almost. considering it's outside, maybe need a sky check or something?

You have to turn on ShadowMaps under DynamicLights section in options. It looks perfectly good.

[Hey Doomer_]

Out of curiosity, what happened to having a compatible version for LZ that i sent you?

On a side note, how do i go about fixing an equivalent for LZ to the errors Unknown identifier 'PPShader' and the error Unknown identifier 'Lightlevel'

Thanks, keep up the fantastic work as this is great.

Holy smokes! I had forgotten about LZ, thanks for reminding me. I've been preoccupied with baked lighting issues.

PPShader is for GZDoom shaders any may just be incompatible. It sets parameters for the shader and would have to be commented out, I would guess.

I'd have to know where you're getting the "Lightlevel" error; this could be related to actors, and that map be something else that is just incompatible with LZ.

Yes i had it commented out but was hoping for a better solution that is "good enough" unfortunately

The Lightlevel error is at around 496 if i remember correct in the hd_actors.zs at the part where it has the caster.Lightlevel = int( etc. etc.

That's the code that casts the actor in shadow. I don't think there's a workaround at this time other than commenting out.

[Hey Doomer_]

map12 is a little bit dark outside. infact it's pitch black almost. considering it's outside, maybe need a sky check or something?

I don't think light level or color map are changed for sector 4 that encircles the map. The light level is 144, note. This might be the sidedef baking shading sidedefs away from the light source (one per sector) that makes it look darker than it is... but 144 is pretty dark.

For now try turning down SubLights under Settings to see if that makes a difference.

[Hey Doomer_]

For some reason the combination of this mod and the mildoom 3 mod cause the framerate to steady plummet all the way down to single digits on map02 of doom 2.

No idea why, just thought you might like to know. Loving the mod otherwise

Interesting. I don't see that when added mildoom 3. Exactly when and where does that happen?

Thanks!

[hitmanx]

huh, that's odd. It's near the little jump with all the shotgunners is when it really goes down.

I do run reshade so maybe the .dll is messing with it somehow, I'll test

[Hey Doomer_]

huh, that's odd. It's near the little jump with all the shotgunners is when it really goes down.

I do run reshade so maybe the .dll is messing with it somehow, I'll test

Thanks, I'll check it out

[thugsta]

Holy smokes! I had forgotten about LZ, thanks for reminding me. I've been preoccupied with baked lighting issues.

PPShader is for GZDoom shaders any may just be incompatible. It sets parameters for the shader and would have to be commented out, I would guess.

I'd have to know where you're getting the "Lightlevel" error; this could be related to actors, and that map be something else that is just incompatible with LZ.

Yes i had it commented out but was hoping for a better solution that is "good enough" unfortunately

The Lightlevel error is at around 496 if i remember correct in the hd_actors.zs at the part where it has the caster.Lightlevel = int( etc. etc.

That's the code that casts the actor in shadow. I don't think there's a workaround at this time other than commenting out.

After you fix what your currently looking at, are you going to dive into the LZ version also?

On a side note, can you repost the archive (or let me access somehow in a mega/mediafire link) up till 4.01b so i can see where up to the last version i can make LZ compatible with (at least with my limited understanding of code)

Thanks as always

[Hey Doomer_]

After you fix what your currently looking at, are you going to dive into the LZ version also?

On a side note, can you repost the archive (or let me access somehow in a mega/mediafire link) up till 4.01b so i can see where up to the last version i can make LZ compatible with (at least with my limited understanding of code)

Thanks as always

Sorry what is it that you're asking for?

My own versioning is all over the place - all the code is being worked on at once - but I'll see what I can do to help.