Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000733Zandronum[All Projects] Bugpublic2012-03-28 11:032018-09-30 23:10
ReporterEdward-san 
Assigned ToTorr Samaho 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version98d 
Target Version2.0Fixed in Version2.0 
Summary0000733: software 3d floors implementation broke lava surface texture
DescriptionWhen I was testing the latest beta build in the Grandvoid jumpmaze server I found out a bug with the lava texture in map 37, which happens also offline.

Just load the demo with this command:

./skulltag -iwad doom2.wad -file jumpmaze2_v6a.pk3 -warp 37 -playdemo texturebug.lmp

Before the lava texture appeared correctly in all the areas. Apparently now it's attached to the deep surface.
The curious thing happens in the last area: some parts of the lava are rendered correctly, but the others not.

I was able to reproduce that in the changeset d6d769c4f5ed + the missing r_3dfloors.cpp/h, but not in the previous one.

Is it a map error?
Attached Filespng file icon Screenshot_Doom_20120328_121103.png [^] (88,450 bytes) 2012-03-28 11:03


png file icon Screenshot_Doom_20120328_121639.png [^] (74,984 bytes) 2012-03-28 11:04


? file icon texturebug.lmp [^] (12,280 bytes) 2012-03-28 11:05
? file icon jumpmaze_bug_test_01.wad [^] (172,054 bytes) 2012-03-29 23:26
? file icon lava_example.wad [^] (9,276 bytes) 2014-12-31 21:38

- Relationships
related to 0000487closed Stronghold "DecorationShip" displayed through ceilings/walls in the software renderer 

-  Notes
User avatar (0002958)
Edward-san (developer)
2012-03-28 11:08
edited on: 2012-03-28 11:09

 Also, it's appreciated if someone can make an example wad from this, because I'd like to test it with zdoom latest svn.
User avatar (0003023)
unknownna (updater)
2012-03-29 23:26

 It took me a while to create the example WAD, but nevertheless, here it is (MAP01). It's a ZDoom issue since it's broken in the latest SVN build (3491) as well.
User avatar (0003024)
Edward-san (developer)
2012-03-29 23:54

 Done.
User avatar (0003117)
Edward-san (developer)
2012-04-03 00:00

 Sorry to ask you again, unknownna, but can you make also an example wad based on the first example wad with a new area based on the information I added here:'http://forum.zdoom.org/viewtopic.php?f=2&t=32385#p614227 [^]' ?
Also, can you add in the same wad the area shown in the second screenshot here? That's because imho this weird behaviour is more interesting than the one shown in the previous screenshot.
User avatar (0003470)
Edward-san (developer)
2012-04-27 11:54

 Fixed:'http://zdoom.org/Changelog/3600/files [^]' but I'm not sure how it can be backported to skulltag.
User avatar (0003662)
Torr Samaho (administrator)
2012-06-02 11:12

 This backport requires numerous other backports that go back to at least ZDoom revision 1600. I will have a look whether it's feasible to do these backports or not later, but first we need to get the first Zandronum beta build out.
User avatar (0011042)
Torr Samaho (administrator)
2014-12-07 18:51

 It was feasible to backport the fix now, so I did.
User avatar (0011043)
cobalt (updater)
2014-12-07 18:55
edited on: 2014-12-07 18:58

 Issue addressed by commit 5b5b95b: out of sequence fix backport from ZDoom revision 3600: - Fixed: sector_t::GetHeightSec() was too aggressive about not returning heightsec.
Committed by Benjamin Berkels [Torr Samaho] on Sunday 07 December 2014 19:45:10

Changes in files:
 src/p_sectors.cpp | 26 ++++++++++++++++++++++++++
 src/r_defs.h | 10 ++--------
 2 files changed, 28 insertions(+), 8 deletions(-)
User avatar (0011158)
Edward-san (developer)
2014-12-31 16:22
edited on: 2014-12-31 21:39

 Actually, the fix is partially complete, because if you load the example wad with latest zandronum 2.0 beta, you'll see the surface darkened. I'll check if it happens in that zdoom commit containing the fix.

[edit]weird, it doesn't happen with that zdoom commit...

[edit2]okay, I tested the revision before the backport, and I'm sure that during the long backport period, something broke the transfer brightness... should I make a new bug report?

[edit3]after hours of compiling and fixing compiling errors, I found that the issue started from d8339813af1c . I used lava_example.wad , attached here, to determine this.
User avatar (0011172)
Arco (updater)
2015-01-01 18:13

 Issue fixed in r150101-1703.
User avatar (0011180)
Edward-san (developer)
2015-01-01 18:26
edited on: 2015-01-01 18:27

 Well, reported the problem in comment'http://zandronum.com/tracker/view.php?id=733#c11158 [^]' over ticket 0002041 .

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2012-03-28 11:03 Edward-san New Issue
2012-03-28 11:03 Edward-san File Added: Screenshot_Doom_20120328_121103.png
2012-03-28 11:04 Edward-san File Added: Screenshot_Doom_20120328_121639.png
2012-03-28 11:05 Edward-san File Added: texturebug.lmp
2012-03-28 11:08 Edward-san Note Added: 0002958
2012-03-28 11:09 Edward-san Note Edited: 0002958 View Revisions
2012-03-28 11:10 unknownna Relationship added related to 0000487
2012-03-29 23:26 unknownna Note Added: 0003023
2012-03-29 23:26 unknownna File Added: jumpmaze_bug_test_01.wad
2012-03-29 23:27 unknownna Status new => confirmed
2012-03-29 23:54 Edward-san Note Added: 0003024
2012-04-03 00:00 Edward-san Note Added: 0003117
2012-04-27 11:54 Edward-san Note Added: 0003470
2012-06-02 11:12 Torr Samaho Note Added: 0003662
2012-06-09 13:22 Torr Samaho Category General => Bug
2012-07-31 19:26 Torr Samaho Status confirmed => acknowledged
2012-07-31 19:26 Torr Samaho Resolution open => backport later
2014-12-07 18:51 Torr Samaho Note Added: 0011042
2014-12-07 18:51 Torr Samaho Assigned To => Torr Samaho
2014-12-07 18:51 Torr Samaho Status acknowledged => needs testing
2014-12-07 18:55 cobalt Target Version => 2.0
2014-12-07 18:55 cobalt Description Updated View Revisions
2014-12-07 18:55 cobalt Note Added: 0011043
2014-12-07 18:56 Dusk Note Edited: 0011043 View Revisions
2014-12-07 18:58 Dusk Note Edited: 0011043 View Revisions
2014-12-07 18:58 Dusk Note Edited: 0011043 View Revisions
2014-12-31 16:22 Edward-san Note Added: 0011158
2014-12-31 16:35 Edward-san Note Edited: 0011158 View Revisions
2014-12-31 17:23 Edward-san Note Edited: 0011158 View Revisions
2014-12-31 17:23 Edward-san Note Edited: 0011158 View Revisions
2014-12-31 17:28 Edward-san Note Edited: 0011158 View Revisions
2014-12-31 21:17 Edward-san Note Edited: 0011158 View Revisions
2014-12-31 21:38 Edward-san File Added: lava_example.wad
2014-12-31 21:39 Edward-san Note Edited: 0011158 View Revisions
2015-01-01 18:13 Arco Note Added: 0011172
2015-01-01 18:13 Arco Status needs testing => resolved
2015-01-01 18:13 Arco Fixed in Version => 2.0
2015-01-01 18:13 Arco Resolution backport later => fixed
2015-01-01 18:26 Edward-san Note Added: 0011180
2015-01-01 18:27 Edward-san Note Edited: 0011180 View Revisions
2015-01-01 18:27 Edward-san Note Edited: 0011180 View Revisions
2018-09-30 23:10 Blzut3 Status resolved => closed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker