Automatic merge of T1.6-rc2-41-gacd714487 and 8 pull requests

openrails-bot · openrails-bot · commit d5294b797a0d · 2025-07-01T20:22:30.000+01:00
- Pull request #1104 at f15bb6b: Handle simple adhesion within the axle module - Pull request #1057 at 1c2bcb4: Switchable brake system - Pull request #1086 at e10390b: Add Settings Exporter tool (copy settings to INI, etc) - Pull request #1091 at 2391bc0: Automatic speed control - Pull request #1110 at 387388e: Fix Activity Runner persists after loading exception - Pull request #1115 at 270f22f: Do not activate ETS switch if no suitable cars are attached - Pull request #1082 at 5845a1a: Allow variable water level in glass gauge - Pull request #1081 at 689494b: Brake cuts power unification
diff --git a/Source/Installer/Installer.iss b/Source/Installer/Installer.iss
@@ -20,7 +20,7 @@
 
 #define DotNETName "Microsoft .NET Framework 4.7.2"
 
-#define MyAppURL "https://openrails.org"
+#define MyAppURL "http://openrails.org" ; Not yet HTTPS
 #define MyAppSourceURL "http://openrails.org/download/source/"
 #define MyAppSupportURL "https://launchpad.net/or"
 
@@ -31,7 +31,7 @@
 #define MyAppDocPath "..\..\Program\Documentation"
 
 #define NetRedistPath "..\..\.NET Framework 4.7.2 web installer"
-#define NetRedist "ndp472-kb4054531-web.exe" ; Has to be lower-case to match download
+#define NetRedist "NDP472-KB4054531-Web.exe"
 
 [Setup]
 ; NOTE: The value of AppId uniquely identifies this application.
@@ -50,26 +50,20 @@ DefaultDirName  ={commonpf}\{#MyAppName}
 DefaultGroupName={#MyAppName}
 AllowNoIcons    =yes
 LicenseFile     ={#MyAppProgPath}\Copying.txt
-InfoBeforeFile  ={#MyAppProgPath}\Prerequisites.txt
+InfoBeforeFile   ={#MyAppProgPath}\Prerequisites.txt
 InfoAfterFile   ={#MyAppProgPath}\Readme.txt
 
-; Prompt for a destination folder
-DisableDirPage  =no
-; 32-bit is the default, installing in "Program Files (x86)" on 64-bit Windows. "x64compatible" uses Program Files on 64-bit Windows
-ArchitecturesInstallIn64BitMode =x64compatible
-
-; Default is admin install mode.
-; Comment in the following line to run in non-administrative install mode, but that cannot create the directory C:\Program Files\Open Rails
-; PrivilegesRequired=lowest
+; Remove the following line to run in administrative install mode (install for all users.)
+; PrivilegesRequired=lowest ; Cannot create the directory C:\Program Files\Open Rails
 
 Compression     =lzma
 SolidCompression=yes
-WizardStyle     =modern
+WizardStyle=modern
 Uninstallable   =yes
-UninstallDisplayIcon ={app}\{#MyAppExeName}
-OutputBaseFilename =OpenRailsSetup
+UninstallDisplayIcon={app}\{#MyAppExeName}
+OutputBaseFilename=OpenRailsSetup
 
-; Windows 7 SP1 Correct value, so ignore final warning message from Build > Compile
+; Windows 7 SP1
 MinVersion      =6.1sp1
 
 [Languages]
@@ -85,7 +79,7 @@ Name: "finnish"; MessagesFile: "compiler:Languages\Finnish.isl"
 Name: "french"; MessagesFile: "compiler:Languages\French.isl"
 Name: "german"; MessagesFile: "compiler:Languages\German.isl"
 Name: "hebrew"; MessagesFile: "compiler:Languages\Hebrew.isl"
-;Name: "icelandic"; MessagesFile: "compiler:Languages\Icelandic.isl"
+Name: "icelandic"; MessagesFile: "compiler:Languages\Icelandic.isl"
 Name: "italian"; MessagesFile: "compiler:Languages\Italian.isl"
 Name: "japanese"; MessagesFile: "compiler:Languages\Japanese.isl"
 Name: "norwegian"; MessagesFile: "compiler:Languages\Norwegian.isl"
diff --git a/Source/Installer/Using the installer scripts.txt b/Source/Installer/Using the installer scripts.txt
@@ -1,24 +1,19 @@
 Using the installer scripts.txt
-Last updated: 2025-06-21 by Chris Jakeman
+Last updated: 2022-08-21 by Chris Jakeman
 
-Compiling the scripts and data files on your own PC and testing them there or on a virtual machine
-==================================================================================================
-
-1. If you wish to update the version name from "1.6", then edit the line
-     #define MyAppVersion "1.6"
+1. If you wish to update the version name from "1.5", then edit the line
+     #define MyAppVersion "1.5"
 	 in the file 
    Source\Installer\Version.iss
      and update the Release number and date in:
    Source\RunActivity\Readme.txt
      and in
    Source\RunActivity\Prerequisites.txt
    
-   The text in these documents is shown by the Installer to the user, so check this is still up to date.
-   
 2. Download and install Inno Setup from http://www.jrsoftware.org/isdl.php
-Initial trials (Apr 2014) used v5.5.4. Open Rails v1.6 uses 6.4.3
+Initial trials (Apr 2014) used v5.5.4. Open Rails v1.5 uses 6.1.2
 
-3. Download Microsoft .NET Framework 4.7.2 web installer package for Windows from:
+3. Download Microsoft .NET Framework 4.7.2 web installer for Windows from:
     https://support.microsoft.com/en-us/topic/microsoft-net-framework-4-7-2-web-installer-for-windows-dda5cddc-b85e-545d-8d4a-d213349b7775
    and save in
   <A>\.NET Framework 4.7.2 web installer\ndp472-kb4054531-web.exe
@@ -28,31 +23,28 @@ Initial trials (Apr 2014) used v5.5.4. Open Rails v1.6 uses 6.4.3
    From Source\Installer, copy OpenRails.iss and Version.iss into <A>\Temp\Installer
   
 5. Repeat the following as often as required:
-  A. Download OR zip file from https://github.com/openrails/openrails/releases, choose the latest (i.e. pre-release) version, open the Assets concertina and pick Open Rails <version>.zip where <version> is something like "1.6-rc2"
+  A. Download OR zip file from https://github.com/openrails/openrails/releases, choose the latest (i.e. pre-release) version, open the Assets concertina and pick OpenRails-Stable.zip
 
   B. Unzip into folder 
-    <A>\Program\
+    <A>\Open Rails\Program\
 
   C. Remove 
-    <A>\Program\Updater.ini 
+    <A>\Open Rails\Program\Updater.ini 
   as James Ross advises because the updater cannot update an installed version yet.
 
-  D. Download source from https://github.com/openrails/openrails/releases, choose the latest (i.e. pre-release) version, open the Assets concertina and pick Source code Open Rails <version> Source.zip where <version> is something like "1.6-rc2" and extract the folder Source\Documentation.
+  D. Download source from https://github.com/openrails/openrails/releases, choose the latest (i.e. pre-release) version, open the Assets concertina and pick Source code (openrails-1.5-rc1.zip) and extract the folder Source\Documentation.
 
   E. Copy contents into folder
-    <A>\Program\Documentation\
-	
-  F. If you made changes to Source\RunActivity\Readme.txt or Source\RunActivity\Prerequisites.txt in Step 1, then
-  copy them to <A>\Program\ to overwrite the downloaded files.
+    <A>\Open Rails\Program\Documentation\
 
-  G. Use Inno Setup Compiler > Build > Compile to compile the installer script
+  F. Use Inno Setup > Build > Compile to compile the installer script
     <A>\Temp\Installer\Installer.iss
 	and generate the Open Rails Installer
   
-  H. The generated file will be found in <A>\Temp\Installer\Output\OpenRailsSetup.exe
+  G. The generated file will be found in <A>\Temp\Installer\Output\OpenRailsSetup.exe
  
 6. Test the installer by building Virtual Machines with a fresh copy of Windows.
-Open Rails v1.6 is compatible with Windows from 7 (SP1) to 11 and, hopefully, beyond.
+Open Rails v1.5 is compatible with Windows from 7 (SP1) to 11 and, hopefully, beyond.
 
 Arrange for the VM to have Internet access and a shared folder.
 Copy OpenRailsSetup.exe to the shared folder.
diff --git a/Source/Installer/Version.iss b/Source/Installer/Version.iss
@@ -1 +1 @@
-#define MyAppVersion "1.6"
+#define MyAppVersion "none"
diff --git a/Source/Orts.Simulation/Simulation/Activity.cs b/Source/Orts.Simulation/Simulation/Activity.cs
@@ -884,8 +884,6 @@ public override void NotifyEvent(ActivityEventType EventType)
                         double sinceActArriveS = (new DateTime().Add(TimeSpan.FromSeconds(Simulator.ClockTime))
                                                 - ActArrive).Value.TotalSeconds;
                         BoardingEndS -= sinceActArriveS;
-
-                        ldbfevaldepartbeforeboarding = false; // reset flag. Debrief Eval
                     }
                     else
                     {
@@ -902,30 +900,29 @@ public override void NotifyEvent(ActivityEventType EventType)
                         if (BoardingS > 0 || ((double)(SchDepart - SchArrive).TotalSeconds > 0 &&
                             MyPlayerTrain.PassengerCarsNumber == 1 && MyPlayerTrain.Cars.Count > 10 ))
                         {
-                            // accepted station stop because either freight train or passenger train or fake passenger train with passenger car on platform or fake passenger train
+                        // accepted station stop because either freight train or passenger train or fake passenger train with passenger car on platform or fake passenger train
                             // with Scheduled Depart > Scheduled Arrive
-                            // ActArrive is usually same as ClockTime
-                            BoardingEndS = Simulator.ClockTime + BoardingS;
+                                // ActArrive is usually same as ClockTime
+                                BoardingEndS = Simulator.ClockTime + BoardingS;
 
-                            if (ActArrive == null)
-                            {
-                                ActArrive = new DateTime().Add(TimeSpan.FromSeconds(Simulator.ClockTime));
-                            }
+                                if (ActArrive == null)
+                                {
+                                    ActArrive = new DateTime().Add(TimeSpan.FromSeconds(Simulator.ClockTime));
+                                }
 
-                            arrived = true;
-                            // But not if game starts after scheduled arrival. In which case actual arrival is assumed to be same as schedule arrival.
-                            double sinceActArriveS = (new DateTime().Add(TimeSpan.FromSeconds(Simulator.ClockTime))
-                                                    - ActArrive).Value.TotalSeconds;
-                            BoardingEndS -= sinceActArriveS;
-                            double SchDepartS = SchDepart.Subtract(new DateTime()).TotalSeconds;
-                            BoardingEndS = CompareTimes.LatestTime((int)SchDepartS, (int)BoardingEndS);
+                                arrived = true;
+                                // But not if game starts after scheduled arrival. In which case actual arrival is assumed to be same as schedule arrival.
+                                double sinceActArriveS = (new DateTime().Add(TimeSpan.FromSeconds(Simulator.ClockTime))
+                                                        - ActArrive).Value.TotalSeconds;
+                                BoardingEndS -= sinceActArriveS;
+                                double SchDepartS = SchDepart.Subtract(new DateTime()).TotalSeconds;
+                                BoardingEndS = CompareTimes.LatestTime((int)SchDepartS, (int)BoardingEndS);
 
+                            }
                         }
-
-                        ldbfevaldepartbeforeboarding = false; // reset flag. Debrief Eval
-                    }
                     if  (MyPlayerTrain.NextSignalObject[0] != null)
                            distanceToNextSignal =  MyPlayerTrain.NextSignalObject[0].DistanceTo(MyPlayerTrain.FrontTDBTraveller);
+
                 }
             }
             else if (EventType == ActivityEventType.TrainStart)
@@ -940,15 +937,6 @@ public override void NotifyEvent(ActivityEventType EventType)
                     if (MyPlayerTrain.TrainType != Train.TRAINTYPE.AI_PLAYERHOSTING)
                        MyPlayerTrain.ClearStation(PlatformEnd1.LinkedPlatformItemId, PlatformEnd2.LinkedPlatformItemId, true);
 
-                    // Debrief Eval: departure before boarding completed
-                    if (!maydepart && !ldbfevaldepartbeforeboarding)
-                    {
-                        var train = Simulator.PlayerLocomotive.Train;
-                        ldbfevaldepartbeforeboarding = true;
-                        DbfEvalDepartBeforeBoarding.Add(PlatformEnd1.Station);
-                        train.DbfEvalValueChanged = true;
-                    }
-
                     if (LogStationStops)
                     {
                         StringBuilder stringBuild = new StringBuilder();
@@ -993,6 +981,15 @@ public override void NotifyEvent(ActivityEventType EventType)
                     {
                         DisplayMessage = Simulator.Catalog.GetStringFmt("Passenger boarding completes in {0:D2}:{1:D2}",
                             remaining / 60, remaining % 60);
+
+                        //Debrief Eval
+                        if (Simulator.PlayerLocomotive.SpeedMpS > 0 && !ldbfevaldepartbeforeboarding)
+                        {
+                            var train = Simulator.PlayerLocomotive.Train;
+                            ldbfevaldepartbeforeboarding = true;
+                            DbfEvalDepartBeforeBoarding.Add(PlatformEnd1.Station);
+                            train.DbfEvalValueChanged = true;
+                        }
                     }
                     // May depart
                     else if (!maydepart)
@@ -1011,6 +1008,8 @@ public override void NotifyEvent(ActivityEventType EventType)
                             if (MyPlayerTrain.IsActualPlayerTrain) Simulator.SoundNotify = Event.PermissionToDepart;
                         }
 
+                        ldbfevaldepartbeforeboarding = false;//reset flag. Debrief Eval
+
                         // if last task, show closure window
                         // also set times in logfile
 
diff --git a/Source/Orts.Simulation/Simulation/Physics/Train.cs b/Source/Orts.Simulation/Simulation/Physics/Train.cs
@@ -1628,7 +1628,6 @@ public void DPMoveToFront()
             if (LeadLocomotive == null || (LeadLocomotive as MSTSLocomotive).DPDynamicBrakeController == null)
                 return;
             int idToMove = -1;
-            bool stillHasRemotes = false;
             for (var i = 0; i < Cars.Count; i++)
             {
                 if (!(Cars[i] is MSTSLocomotive))
@@ -1642,12 +1641,7 @@ public void DPMoveToFront()
                     Cars[i].RemoteControlGroup = 0;
                 else if (idToMove > -1 && Cars[i].RemoteControlGroup == 0)
                     Cars[i].RemoteControlGroup = 1;
-
-                if (Cars[i].RemoteControlGroup == 1)
-                    stillHasRemotes = true;
             }
-            if (!stillHasRemotes)
-                DPMode = 0;
         }
 
         /// <summary>
@@ -1662,7 +1656,6 @@ public void DPMoveToBack()
             var dpDynamicBrakeCurrentNotch = MathHelper.Clamp((LeadLocomotive as MSTSLocomotive).DPDynamicBrakeController.GetNotch(dpDynamicBrakePercent/100), 0, 8);
             var dpThrottleCurrentNotch = (LeadLocomotive as MSTSLocomotive).ThrottleController.CurrentNotch;
             int idToMove = -1;
-            bool alreadyHasRemotes = false;
             int idLead = LeadLocomotive != null ? (Cars[LeadLocomotiveIndex] as MSTSLocomotive).DPUnitID : -1;
             for (var i = Cars.Count - 1; i >= 0; i--)
             {
@@ -1674,7 +1667,6 @@ public void DPMoveToBack()
                     dpThrottlePercent = DPThrottlePercent;
                     dpDynamicBrakeCurrentNotch = (LeadLocomotive as MSTSLocomotive).DPDynamicBrakeController.CurrentNotch;
                     dpThrottleCurrentNotch = (LeadLocomotive as MSTSLocomotive).DPThrottleController.CurrentNotch;
-                    alreadyHasRemotes = true;
                     continue;
                 }
                 if (idToMove == -1 && Cars[i].RemoteControlGroup == 0)
@@ -1694,13 +1686,6 @@ public void DPMoveToBack()
                 else if (idToMove > -1 && Cars[i].RemoteControlGroup == 1)
                     Cars[i].RemoteControlGroup = 0;
             }
-            if (!alreadyHasRemotes)
-            {
-                if (LeadLocomotive.ThrottlePercent > 0)
-                    DPMode = 1;
-                else if (LeadLocomotive.DynamicBrakePercent >= 0)  // off is -1
-                    DPMode = -1;
-            }
         }
 
         /// <summary>
diff --git a/Source/RunActivity/Prerequisites.txt b/Source/RunActivity/Prerequisites.txt
@@ -1,6 +1,7 @@
-﻿Open Rails software Prerequisites.txt - Release v1.6 June 2025
+﻿Open Rails software Prerequisites.txt - Release v1.5
+September 2022
 
-Visit our web-page at https://openrails.org for details and links.
+Visit our web-page at http://openrails.org for details and links.
 
 Please note the distribution and use of Open Rails software is governed by the GNU License - see file Copying.txt. 
 Source code can be downloaded from https://github.com/openrails/openrails
diff --git a/Source/RunActivity/Readme.txt b/Source/RunActivity/Readme.txt
@@ -1,23 +1,29 @@
-﻿Open Rails software Readme.txt - Release v1.6 - June 2025
+﻿Open Rails software Readme.txt - Release v1.5
+September 2022
 
-Visit our web-page at https://openrails.org for details and links.
+Visit our web-page at http://openrails.org for details and links.
 
-Please note the distribution and use of Open Rails software is governed by the GNU License - see file Copying.txt. Source code can be downloaded from https://github.com/openrails/openrails
+Please note the distribution and use of Open Rails software is governed by the GNU License - see file Copying.txt. 
+Source code can be downloaded from https://github.com/openrails/openrails
 
 OPERATION
-- You can run Open Rails by clicking on the Desktop icon or the Quick Launch icon or the Start Menu entry or C:\Program Files\Open Rails\OpenRails.exe
+- You can run Open Rails by clicking on the Desktop icon or the Quick Launch icon or the Start Menu entry or C:\Program Files (x86)\Open Rails\OpenRails.exe
 - Help and Game Controls are available in-game using the Help key (F1).
 - Please read the operations manual in Open Rails > Documents > Manual.pdf
 
 CONTENT
 This Open Rails download does not include any content - no routes, trains, activities - just the simulation program. 
 
-If you have content suitable for Open Rails or Microsoft Train Simulator already in place, then you can use the Open Rails program to operate those routes and drive those trains straight away. If not, then you will need to install some models bought from a vendor or free from the community before you can use Open Rails.
+If you have content suitable for Open Rails or Microsoft Train Simulator already in place, then you can use the
+Open Rails program to operate those routes and drive those trains straight away.
 
-Use the Content button for some auto-install routes or visit the catalogue at www.openrails.org/download/explore-content
+If not, then you will need to install some models bought from a vendor or free from the community before you can use Open Rails.
+
+Or you can try out the 2 self-installing models on our home page - both free of charge.
 
 SUPPORT
 Please send bug reports and feature requests to our Bug Tracker at https://launchpad.net/or
 
 UPDATES
-Open Rails checks once a day for new versions and, if found, adds a red marker to the notification icon in the top right of the menu form. Click on the notification to find the details of an update and install it.
+Open Rails checks once a day for new versions and, if found, adds an Update link in the top right of its main menu. 
+Click this Update link to download and update your Open Rails.
diff --git a/Source/RunActivity/Viewer3D/AnimatedPart.cs b/Source/RunActivity/Viewer3D/AnimatedPart.cs
@@ -130,16 +130,9 @@ public void SetFrameCycle(float frame)
         public void SetFrameWrap(float frame)
         {
             // Wrap the frame around 0-MaxFrame without hanging when MaxFrame=0.
-            if (MaxFrame > 0)
-            {
-                frame %= MaxFrame;
-                // If frame was negative (eg: animation run in reverse), it will still be negative
-                // and needs one additional offset by MaxFrame to be in the correct range
-                if (frame < 0)
-                    frame += MaxFrame;
-            }
-            else if (frame < 0)
-                frame = 0;
+            while (MaxFrame > 0 && frame < 0) frame += MaxFrame;
+            if (frame < 0) frame = 0;
+            frame %= MaxFrame;
             SetFrame(frame);
         }
 
diff --git a/Source/RunActivity/Viewer3D/Lights.cs b/Source/RunActivity/Viewer3D/Lights.cs

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-#define MyAppVersion "1.6"`
	`1`	`+#define MyAppVersion "none"`