Brightmaps Plus v1.7

For high-res texture/sprite projects, sprite-fix patches, music add-ons, music randomizers, and other graphic/sound-only projects.
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 (especially 3DGE) are perfectly acceptable here too.

Please read the full rules for more details.

Re: Brightmaps Plus (spritefix compatible)

Postby mgtroyas » Thu Dec 19, 2019 4:20 pm

Hi NightFright!

I've created an issue on your github, but I'll post here just in case.

I've been using may version of your brightmaps on many mods without an issue. I today updated to latest version and in my first play I see this weird effect on a slime waterfall on map23 of Back to Saturn X E1.

Perhaps it also happens on other mods, I haven't tested yet.

User avatar
mgtroyas
 
Joined: 25 Mar 2017

Re: Brightmaps Plus (spritefix compatible)

Postby NightFright » Fri Dec 20, 2019 2:40 am

Please hold back with error reports until GZD 4.2.5. It will address an issue with custom texture replacements which shouldn't use these brightmaps at all (when using the "iwad" keyword). It's not a problem with the mod itself.

As an explanation: I defined all texture brightmaps so that they wouldn't be used as soon as a mod replaces the original lump, avoiding situations where a replacement texture would be different from the original while the old brightmap would still be used. However, due to a GZD bug, the definitions don't work as intended. It was reported and fixed after the 4.2.4 release.

Also check my post from Nov 5 in this thread for comparison.

If you don't like to wait, you can try running the mod with a recent development snapshot.
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus (spritefix compatible)

Postby Agitatio » Fri Dec 20, 2019 5:05 am

Does that mean that brightmaps will finally be able to be autoloaded without fear of having them show up on textures they weren't supposed to? That always stopped me from doing so, even though I wanted to. Sorry if that question is a bit offtopic.
User avatar
Agitatio
 
Joined: 05 Sep 2011
Discord: Agitatio#4183
Operating System: Windows 10/8.1/8 64-bit
OS Test Version: No (Using Stable Public Version)
Graphics Processor: nVidia with Vulkan support

Re: Brightmaps Plus (spritefix compatible)

Postby NightFright » Fri Dec 20, 2019 5:21 am

In theory it has always been possible to do so if affected brightmaps had the "iwad" keyword in their definitions. I don't know when this GZD feature stopped working, but it should be possible to use it again in any GZD dev build after Nov 5.

I haven't applied the iwad feature for sprites in my mod though since otherwise brightmaps for spritefixes wouldn't be possible.
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus (spritefix compatible)

Postby Kazudra » Tue Feb 04, 2020 4:17 pm

NightFright wrote:In theory it has always been possible to do so if affected brightmaps had the "iwad" keyword in their definitions. I don't know when this GZD feature stopped working, but it should be possible to use it again in any GZD dev build after Nov 5.

I haven't applied the iwad feature for sprites in my mod though since otherwise brightmaps for spritefixes wouldn't be possible.


Removing all instances of "iwad" fixes everything.
Kazudra
 
Joined: 25 May 2015

Re: Brightmaps Plus (spritefix compatible)

Postby NightFright » Wed Feb 05, 2020 1:41 am

Actually it doesn't, and removing iwad lines from the defs would remove one of the most prominent features of the mod. You need to use more recent GZDoom builds, it will work. Already tested and confirmed.
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus (spritefix compatible)

Postby Agitatio » Wed Feb 05, 2020 1:47 pm

What do you think about splitting brightmaps in two, for sprites and textures? So that texture brightmaps can be safely autoloaded.
Or can sprite brightmaps be somehow made toggleable through options?
User avatar
Agitatio
 
Joined: 05 Sep 2011
Discord: Agitatio#4183
Operating System: Windows 10/8.1/8 64-bit
OS Test Version: No (Using Stable Public Version)
Graphics Processor: nVidia with Vulkan support

Re: Brightmaps Plus (spritefix compatible)

Postby NightFright » Wed Feb 05, 2020 2:52 pm

There are two goals remaining for this pack, but unsure if technically possible:

1) Toggle to switch between spritefixes and vanilla
2) Toggle for texture brightmaps

Right now it is not possible to dynamically reload a GLDEFS file without restarting GZDoom, at least AFAIK. If it can be changed, it'd be a nice last thing to polish this.

If it's not going to happen, I guess three packs need to be made:
1) Sprites only, vanilla
2) Sprites only, spritefix
3) Textures only
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus (spritefix compatible)

Postby Kazudra » Wed Feb 05, 2020 10:56 pm

NightFright wrote:Actually it doesn't, and removing iwad lines from the defs would remove one of the most prominent features of the mod. You need to use more recent GZDoom builds, it will work. Already tested and confirmed.

Mmhmm.
Kazudra
 
Joined: 25 May 2015

Re: Brightmaps Plus (spritefix compatible)

Postby Nash » Wed Feb 05, 2020 11:27 pm

Night, the more I think about it, yeah, a separated, modular pack sounds better because it's not possible to toggle this stuff at runtime.
User avatar
Nash
 
 
 
Joined: 27 Oct 2003
Location: Kuala Lumpur, Malaysia
Github ID: nashmuhandes

Version 1.7 released

Postby NightFright » Thu Feb 06, 2020 4:53 am

It is as I thought, then. Well, in that case nothing holds me back from releasing the split versions, I guess! I took the opportunity to fix some issues with texture brightmaps while I was at it. This release should also put an end to illuminated floors and/or ceilings in dark areas (such as Kazudra posted above, even though it had nothing to do with using iwad definitions).

Changelog v1.7 (Feb 6, 2020)
[GENERAL] Split pack into three parts: bmplus_vanilla.pk3, bmplus_spritefix.pk3, bmplus_textures.pk3
[DOOM/DOOM2] Deactivated certain flat brightmaps to improve dark level areas (CEIL1_2, CEIL1_3, CEIL3_4, CEIL3_6, FLAT2, FLAT17, FLOOR1_7, TLITE6_1, TLITE6_5 and GRNLITE1)
[DOOM] Fix for texture brightmaps STEP4 and STEP9
[DOOM] Improved flat brightmap GATE4

If you want to continue with the full experience from previous single-file releases, the following setup is recommended:
- Doom/Doom2/Final Doom: bmplus_vanilla.pk3 (for spritefixes: bmplus_spritefix.pk3), bmplus_textures.pk3
- Heretic: bmplus_vanilla.pk3 (for spritefixes: bmplus_spritefix.pk3), bmplus_textures.pk3
- Hexen: bmplus_vanilla.pk3, bmplus_textures.pk3
- Strife: bmplus_vanilla.pk3
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus v1.7

Postby Agitatio » Thu Feb 06, 2020 11:48 am

I'm confused now. bmplus_vanilla and bmplus_spritefix contain brightmaps for all sprites only, but one is for vanilla and the other one is for spritefix, right? And bmplus_textures is, well, brightmaps for textures. Is that correct?

I wonder though. If before you couldn't use iwad definitions for sprite brightmaps because they wouldn't work with spritefix, why not use it now for vanilla and just bundle sprite and texture brightmaps into one?
User avatar
Agitatio
 
Joined: 05 Sep 2011
Discord: Agitatio#4183
Operating System: Windows 10/8.1/8 64-bit
OS Test Version: No (Using Stable Public Version)
Graphics Processor: nVidia with Vulkan support

Re: Brightmaps Plus v1.7

Postby NightFright » Thu Feb 06, 2020 12:16 pm

To summarize again:
- bm_vanilla: Sprites only, for vanilla users
- bm_spritefix: Sprites only, for spritefixed Doom or Heretic, but also vanilla Hexen+Strife
- bm_textures: Textures only, for Doom, Heretic and Hexen

If you want the exact same thing as before, choose one of the two sprite packs and load it with the textures pack. You can also use them individually, i.e. just sprites or textures only. It all works.

You can combine vanilla sprites+textures and spritefixes+textures into two new compilations, but then there would be people again who don't want texture brightmaps. Anyway, I am also using iwad parameters for vanilla, it's only missing the brightmap deviations required for spritefixes. The reason I split this was done so that people can combine the packs the way they want without having to mess with the internal definitions manually. From what I understood this was the request in general, unless I got it wrong.

Maybe I need to make three more packs, a combined one with everything like before plus the two other combinations mentioned above. Good lord. I knew this would be confusing, that's why I went for a single file solution until now. And something tells me that in the end it will be like that again. Sigh...
Last edited by NightFright on Thu Feb 06, 2020 12:32 pm, edited 1 time in total.
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

Re: Brightmaps Plus v1.7

Postby Agitatio » Thu Feb 06, 2020 12:31 pm

NightFright wrote:You can combine vanilla sprites+textures and spritefixes+textures into two new compilations, but then there would be people again who don't want texture brightmaps. Anyway, I am also using iwad parameters for vanilla, it's only missing the brightmap deviations required for spritefixes. The reason I split this was done so that people can combine the packs the way they want without having to mess with the internal definitions manually. From what I understood this was the request in general, unless I got it wrong.

I see. So sprites+textures are now 100% autoload safe? And spritefixes should be kept to be used when nothing overrides sprites?

NightFright wrote:Maybe I need to make three more packs, a combined one with everything like before plus the two other combinations mentioned above. Good lord. I knew this would be confusing, that's why I went for a single file solution until now. And something tells me that in the end it will be like that again. Sigh...

Nah, I get why it was split like that now. But maybe better names for files or some short description in the OP could help with possible future confusion.
Something like bmplus_sprites and bmplus_textures for vanilla sprites and textures, and bmplus_sprites_spfx for SPFX?
User avatar
Agitatio
 
Joined: 05 Sep 2011
Discord: Agitatio#4183
Operating System: Windows 10/8.1/8 64-bit
OS Test Version: No (Using Stable Public Version)
Graphics Processor: nVidia with Vulkan support

Re: Brightmaps Plus v1.7

Postby NightFright » Thu Feb 06, 2020 12:34 pm

I am opposing the entire autoload idea and prefer a modular approach via launcher where you can conveniently turn mods on and off as needed. That being said, since I am using iwad parameter everywhere, which I had already done before but it didn't work due to a GZD bug, any mod loaded on top of this won't cause problems since any modified sprites simply won't use these brightmaps. If that's what you want, it's autoload ready. And you don't have to add the textures pack if you don't like it.

Anyway, I edited my post above and changed download description in OP, hopefully it's clearer now.
User avatar
NightFright
Spotlight Team
 
Joined: 02 May 2008
Location: Germany

PreviousNext

Return to Graphic/Audio Patches

Who is online

Users browsing this forum: No registered users and 3 guests