Source Code Project Mantis - FSSCP
View Issue Details
0002396FSSCPOpenGLpublic2011-02-09 15:102012-11-29 14:11
Reportercasual pilot 
Assigned ToZacam 
PrioritynormalSeverityminorReproducibilityalways
StatusfeedbackResolutionreopened 
PlatformOSOS Version
Product Version3.6.12 
Target VersionFixed in Version 
Summary0002396: Jump node rendering very slow
DescriptionThe jump node wireframe model renders very slowly, dropping the frame rate to approximately 1 fps.
Additional InformationLinux, r200 DRI driver (Radeon 9250)

I tried setting the MR_SHOW_OUTLINE_HTL flag (see patch). That fixes the speed issue but causes incorrect rendering when the jump node activates.
TagsNo tags attached.
related to 0002676resolved Zacam FRED "Show Outlines" rendering mode exceptionally slow. 
Attached Filespatch fso-jumpy.patch (510) 2011-02-09 15:10
http://scp.indiegames.us/mantis/file_download.php?file_id=1639&type=bug
patch 2396_jumpnode.cpp.patch (426) 2012-07-02 12:58
http://scp.indiegames.us/mantis/file_download.php?file_id=1905&type=bug
patch vajumpnodes.patch (4,097) 2012-11-29 14:08
http://scp.indiegames.us/mantis/file_download.php?file_id=1994&type=bug

Notes
(0012632)
Echelon9   
2011-02-25 22:27   
Can we confirm that this isn't a driver specific issue on Linux?
(0012633)
casual pilot   
2011-02-26 04:45   
I don't have other platforms to test on, and there are no alternative drivers for this video card. However, if this were not a platform specific issue, surely it would have been detected long ago. For some reason the original rendering method is extremely slow on this platform and changing it with the flag causes a remarkable speedup. The speedup is probably less noticable on other platforms.

The rendering errors manifest themselves as flashing brown lines, typically from the center of the screen outwards. They appear also when the node is inactive - I haven't been able to pin down the exact conditions. The green jump node wireframe disappears when the brown lines appear. I am not sure if the rendering errors are platform-specific. Also, because the rendering errors are not easily reproducible and the game is totally unplayable with the original rendering method, I am not sure whether the errors are specific to HTL rendering.
(0012634)
casual pilot   
2011-02-27 07:35   
(Last edited: 2011-02-28 05:00)
Another case of horribly slow rendering that may be related: if -ship_choice_3d is enabled, on the weapon selection screen the lines connecting the selected weapon icon to the weapon locations in the ship model are rendered very slowly.

(0013787)
Goober5000   
2012-07-02 02:31   
The jump nodes are rendered in non-HTL because nobody could get the wireframe HTL rendering to work properly. So this just recapitulates an old, unresolved bug.

Anyone know how to do proper wireframes in HTL?
(0013802)
Zacam   
2012-07-02 12:58   
The supplied patch is -close- but not quite. Instead of adding MR_SHOW_OUTLINE_HTL in addition to the options, it should be used as a replacement for MR_SHOW_OTLINE_PRESET.

Newer patch attached.
(0013806)
Goober5000   
2012-07-03 01:00   
Fix committed to trunk@8975.
(0013822)
Zacam   
2012-07-04 15:49   
Fix committed to fs2_open_3_6_14@8987.
(0013844)
Goober5000   
2012-07-07 17:00   
Reopening this bug because the proposed fix adds diagonal lines to the jump node model. :-/

Hoping casual pilot or another rendering person might be able to assist.
(0014230)
Valathil   
2012-11-29 14:08   
(Last edited: 2012-11-29 14:11)
Attached a patch that might speed up rendering by drawing the nodes with vertex arrays instead of seperate lines. Not a final version, might require some cleanup and additional parameters to not interfere with nohtl, just a quick test to see if its faster.


Issue History
2011-02-09 15:10casual pilotNew Issue
2011-02-09 15:10casual pilotFile Added: fso-jumpy.patch
2011-02-25 22:27Echelon9Note Added: 0012632
2011-02-25 22:27Echelon9Statusnew => feedback
2011-02-26 04:45casual pilotNote Added: 0012633
2011-02-27 07:35casual pilotNote Added: 0012634
2011-02-28 05:00casual pilotNote Edited: 0012634
2012-07-02 02:31Goober5000Note Added: 0013787
2012-07-02 12:58ZacamNote Added: 0013802
2012-07-02 12:58ZacamFile Added: 2396_jumpnode.cpp.patch
2012-07-02 12:58ZacamAssigned To => Zacam
2012-07-02 12:58ZacamStatusfeedback => assigned
2012-07-02 12:59ZacamStatusassigned => feedback
2012-07-02 15:10ZacamStatusfeedback => code review
2012-07-03 01:00Goober5000Changeset attached => fs2open trunk r8975
2012-07-03 01:00Goober5000Note Added: 0013806
2012-07-03 01:00Goober5000Statuscode review => resolved
2012-07-03 01:00Goober5000Resolutionopen => fixed
2012-07-04 15:49ZacamChangeset attached => fs2open fs2_open_3_6_14 r8987
2012-07-04 15:49ZacamNote Added: 0013822
2012-07-07 16:53Goober5000Relationship addedrelated to 0002676
2012-07-07 16:58Goober5000Changeset attached => fs2open trunk r9001
2012-07-07 17:00Goober5000Note Added: 0013844
2012-07-07 17:00Goober5000Statusresolved => feedback
2012-07-07 17:00Goober5000Resolutionfixed => reopened
2012-07-23 22:09ZacamChangeset attached => fs2open fs2_open_3_6_14 r9036
2012-11-29 14:08ValathilFile Added: vajumpnodes.patch
2012-11-29 14:08ValathilNote Added: 0014230
2012-11-29 14:11ValathilNote Edited: 0014230bug_revision_view_page.php?bugnote_id=14230#r257