Brightmaps Plus v1.92
Forum rules
The Projects forums are only for 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.
The Projects forums are only for 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.
-
- Posts: 8
- Joined: Sat Mar 25, 2017 9:06 am
Re: Brightmaps Plus (spritefix compatible)
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.
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.
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus (spritefix compatible)
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.
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.
-
- Posts: 242
- Joined: Mon Sep 05, 2011 10:07 am
- Graphics Processor: nVidia with Vulkan support
Re: Brightmaps Plus (spritefix compatible)
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.
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus (spritefix compatible)
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.
I haven't applied the iwad feature for sprites in my mod though since otherwise brightmaps for spritefixes wouldn't be possible.
-
- Posts: 126
- Joined: Mon May 25, 2015 10:52 am
Re: Brightmaps Plus (spritefix compatible)
Removing all instances of "iwad" fixes everything.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.
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus (spritefix compatible)
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.
-
- Posts: 242
- Joined: Mon Sep 05, 2011 10:07 am
- Graphics Processor: nVidia with Vulkan support
Re: Brightmaps Plus (spritefix compatible)
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?
Or can sprite brightmaps be somehow made toggleable through options?
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus (spritefix compatible)
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
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
-
- Posts: 126
- Joined: Mon May 25, 2015 10:52 am
-
-
- Posts: 17465
- Joined: Mon Oct 27, 2003 12:07 am
- Location: Kuala Lumpur, Malaysia
Re: Brightmaps Plus (spritefix compatible)
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.
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Version 1.7 released
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
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
-
- Posts: 242
- Joined: Mon Sep 05, 2011 10:07 am
- Graphics Processor: nVidia with Vulkan support
Re: Brightmaps Plus v1.7
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?
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?
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus v1.7
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...
- 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.
-
- Posts: 242
- Joined: Mon Sep 05, 2011 10:07 am
- Graphics Processor: nVidia with Vulkan support
Re: Brightmaps Plus v1.7
I see. So sprites+textures are now 100% autoload safe? And spritefixes should be kept to be used when nothing overrides sprites?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.
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.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...
Something like bmplus_sprites and bmplus_textures for vanilla sprites and textures, and bmplus_sprites_spfx for SPFX?
-
- Spotlight Team
- Posts: 1375
- Joined: Fri May 02, 2008 12:29 pm
- Location: Germany
Re: Brightmaps Plus v1.7
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.
Anyway, I edited my post above and changed download description in OP, hopefully it's clearer now.