Refueling process pointing to wrong end of tank.

Bug #1584255 reported by Edward Keenan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Open Rails
In Progress
Undecided
Edward Keenan

Bug Description

This issue is caused by the unnecessary change in the negative/positive value in match.IntakePoint.OffsetM. This was already done by time this was processed in GetDistanceToM(). I should have a fix for this shortly.

Edward K.

Edward Keenan (edwardk)
Changed in or:
assignee: nobody → Edward Keenan (edwardk)
status: New → In Progress
Revision history for this message
Edward Keenan (edwardk) wrote :

Turns out that there is more to this. If the intake value in the engine file is negative, the point where it is processed is positive, but if the intake value is positive, it remains positive. This is why there is a problem with the refueling location.

Revision history for this message
Edward Keenan (edwardk) wrote :

The items below should show the inconsistent manner in how the source OffsetM value is treated.

The first 2 show the OffsetM value from a steam tender. Note that when using the -match, the position would be correct since under OR, the rear of the car is represented with a positive value.

Information: The value of OffsetM applying -match.IntakePoint.OffsetM is: 3.2 (Source is -3.2)

Information: The value of OffsetM applying match.IntakePoint.OffsetM is: -3.2

The next 2 which shows the issue is from a diesel locomotive. The source OffsetM is a positive value which means under OR, it should be negative since this would represent the front portion of a wagon. You would think that using the -match would correct this properly, but in this case it does not. You will notice that the +match creates the proper OffsetM conversion.

Information: The value of OffsetM applying -match.IntakePoint.OffsetM is: 3.33 (Source is 3.33)

Information: The value of OffsetM applying match.IntakePoint.OffsetM is: -3.33

The -match is currently used in the code.

Revision history for this message
Edward Keenan (edwardk) wrote :

Found out what was causing the above issue. Fix will be submitted once I finish testing.

Edward Keenan (edwardk)
Changed in or:
status: In Progress → Fix Released
James Ross (twpol)
Changed in or:
status: Fix Released → In Progress
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.