Signal light disappear since revision #2322

Bug #1346315 reported by BB25187
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Open Rails
Fix Released
Low
James Ross

Bug Description

As explained in a topic of the Elvas Tower forum, we noticed that the light of some signal disappear when one get closer from them. After some back-tracking, it seems that this happens since revision #2322 (Experimental signal light glows). The issue happens when the experimental option is not set. It doesn't affect all signals.
 To reproduce, you may want to (sorry, I couldn't find a simpler setup :-( ):
- Load this route at http://www.activitysimulatorworld.net/chargements_temporaires/Lignes/ASW_Chargements_Ligne_PLM_V2.html. You should install the three files of the version 2 of the route using 7-Zip (PLM_V2.7z.001 to 003), and the version 3 (PLM V3_07.exe
 - Use the path attached to the topic pointed above to create an activity. The first signal you will meet exposes the problem.

I attache a possible patch, which restores the z-bias that was applied until rev #2321 to Out.Position.z. This z-bias was removed at rev #2322, and replaced by non proportional offset, which doesn't work when distance is small. The behavior when glow option is enabled isn't modified by this patch.

Tags: graphics
Revision history for this message
BB25187 (bb25187) wrote :
Revision history for this message
James Ross (twpol) wrote :

It is worth noting that the old positioning would cause the light to incorrectly cut through bits of the signal shape, so I'm not sure we should just go straight back to it.

Revision history for this message
BB25187 (bb25187) wrote :

My experience (with french, german, austrian and swiss) signals is that most of the troubles with old positioning are caused by the (much) larger light size in OR than in MSTS. Reducing the size of the light is often sufficient to avoid nasty interferences between signal shape and lights. This may be different with signals from other countries though. From this stand point, revision #2322 didn't improve things significantly. But clearly the only purpose of this patch was to avoid a regression when the new glow option isn't set.

Revision history for this message
James Ross (twpol) wrote :

I am not aware of this "(much) larger light size in OR than in MSTS" and if that is correct you better file a bug with details, as it should be fixed.

Revision history for this message
BB25187 (bb25187) wrote :

OK James. Sorry for not having reported it already :-(
I will prepare a small setup to expose this in the coming days.

Revision history for this message
BB25187 (bb25187) wrote :

I think my statement on signal light size was wrong, and things are more subtil than I first thought.
I compared the rendering in the two simulators, and in fact the size looks nearly similar. What I interpreted as a larger size was in fact linked to the point you raised on the parts of the signal incorrectly hidden by the lights. This seems to be highly dépendent on the distance and angle of view.
I confirm that from this standpoint, the revision #2322 represents a significant improvement.
As for the light which disappears, I could also resolve it by increasing the light offset to 2.5 cm (instead of 1). This preserves the improvement on light/shape ordering. Nevertheless, this remains somewhat empiric and may not work for other signals (for instance, the wrong ordering reappears when the offset is larger than 4 cm).
Just in case, I attach the tricial patch which corresponds to this last attempt.

James Ross (twpol)
tags: added: graphics
removed: signals
Revision history for this message
James Ross (twpol) wrote :

I've had a look in MSTS and as best I can work out, it puts the signal light 1.5cm in front of the location specified in the file (at least when the camera is close - it's hard to know what it does further away).

Changed in or:
status: New → In Progress
importance: Undecided → Low
assignee: nobody → James Ross (twpol)
status: In Progress → Fix Committed
milestone: none → 1.0
Revision history for this message
James Ross (twpol) wrote :

Based on the MSTS attachment, I've changed Open Rails to use 1.5cm in X2560. This does appear to resolve the problem for me on the PLM route mentioned above.

James Ross (twpol)
Changed in or:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.