Source Code Project Mantis - FSSCP
View Issue Details
0001889FSSCPgraphicspublic2009-02-25 08:392010-12-20 02:46
ReporterEchelon9 
Assigned ToEchelon9 
PrioritynormalSeverityfeatureReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version3.6.9 
Target VersionFixed in Version3.6.13 
Summary0001889: HUD target box doesn't render lighting correctly
DescriptionThe HUD target box in the lower left corner of the screen displays a miniature model of the currently targeted object.

FS2Open does not render lighting on this model, while the rest of the engine does.

Furthermore, a key command (default Shift-Alt-Q) lets you change between 'standard', 'wireframe only' and 'wireframe with textures' rendering in the HUD target box.

In code/hud/hudtargetbox.cpp
----------------------------

    //0==standard
    //1==wireframe only
    //2==wireframe with textures
    Targetbox_wire++;
        if (Targetbox_wire==3)
            Targetbox_wire=0;

----------------------------

This does not work correctly, and I have real doubts as to what "wireframe with textures" would actually look like.
TagsNo tags attached.
Attached Filespatch hudtarget-method_mantis-1889.patch (3,028) 2009-02-25 08:40
http://scp.indiegames.us/mantis/file_download.php?file_id=1244&type=bug

Notes
(0010689)
Echelon9   
2009-02-25 08:40   
Suggested patch has been attached.
(0010690)
taylor   
2009-02-25 09:12   
It was never supposed to use lighting in the targetbox, so that's not a bug. :) I think it has been brought up in the forums before and is always dismissed as a design decision or something, I can't remember exactly.

The wireframe with textures thing is something that I worked on at one point but never could get it working properly with HTL mode. It is basically just outline mode. However, since line drawing HTL models is insanely slow, and the feature is only available as a (non-default) option for the targetbox and in FRED, I quickly gave up on trying to get it working since it just wasn't worth the effort.
(0010691)
Woolie Wool   
2009-02-25 10:56   
Target box miniatures are likely fullbright so the player can see the entire form of the targeted ship in the box instead of just the parts that are lit.
(0010753)
Echelon9   
2009-03-19 23:28   
I'll be leaving this dormant until 3.6.10 is out. I still think it's a good improvement (while not affecting the option to use the current approach), but as a feature it will have to wait until after the code freeze is lifted.
(0011594)
KeldorKatarn   
2010-01-28 13:19   
(Last edited: 2010-01-28 13:20)
This is working as designed. A fully lit version could be added as a 4th option in the target box mode. The standard is also used because fully lit means a lot of render time.

"wireframe with textures" currently doesn't work.

(0012566)
Echelon9   
2010-12-20 02:46   
Resolved / feature in r6870.

The third unused hud target box rendering style has been enable to provide a standard render with proper lighting.

Issue History
2009-02-25 08:39Echelon9New Issue
2009-02-25 08:40Echelon9File Added: hudtarget-method_mantis-1889.patch
2009-02-25 08:40Echelon9Statusnew => assigned
2009-02-25 08:40Echelon9Assigned To => Echelon9
2009-02-25 08:40Echelon9Note Added: 0010689
2009-02-25 09:12taylorNote Added: 0010690
2009-02-25 10:56Woolie WoolNote Added: 0010691
2009-03-19 23:28Echelon9Note Added: 0010753
2009-03-19 23:29Echelon9Resolutionopen => suspended
2009-06-04 10:13Echelon9Statusassigned => feedback
2010-01-27 00:10Echelon9Severitytweak => feature
2010-01-28 13:19KeldorKatarnNote Added: 0011594
2010-01-28 13:20KeldorKatarnNote Edited: 0011594
2010-12-20 02:46Echelon9Note Added: 0012566
2010-12-20 02:46Echelon9Statusfeedback => resolved
2010-12-20 02:46Echelon9Fixed in Version => 3.6.13
2010-12-20 02:46Echelon9Resolutionsuspended => fixed