Merge pull request #617 from cjakeman/menu-options02m

02m: Removed menu option Debrief Evaluation. Now always available.

Author: cjakeman

Source/Documentation/Manual/driving.rst

@@ -633,7 +633,7 @@ within the cabview, see :ref:`here <cabs-odometer>`.
 .. _debriefeval:
 
 
-Debrief Evaluation
+Activity Evaluation
 ------------------
 
 ``Description``
@@ -643,15 +643,14 @@ This feature displays a real-time evaluation of the player's performance
 during the activity run and a final report at the end of an activity.
 The evaluation reports various parameters to provide to the player info in order 
 to improve his train driving ability.
-While the activity is running relevant data are stored and displayed.
-The stored data are used to generate a report at the end of the activity.
+While the activity is running, relevant data are stored and displayed.
+The stored data is used to generate a report at the end of the activity.
 
 
 ``How It Works``
 ''''''''''''''''
 
-Activity evaluation is enabled only for Activity mode, and requires the 
-"Debrief evaluation" checkbox in the main menu window to be enabled.
+Activity evaluation is enabled only for Activity mode.
 Checking some checkboxes within the various option tabs of the main menu 
 provides additional parameters for the activity evaluation. 
 
@@ -664,23 +663,11 @@ and here an example about the Options/Simulation tab:
 .. image:: images/dbfeval-options-simulation.png
 
 Checkboxes shown as unchecked in the two above pictures may be checked or 
-unchecked, but don't have effect to activity evaluation.
+unchecked, but don't have any effect on activity evaluation.
 
 A tab named "Evaluation" is present on the F1 Help Information Monitor.
 Once the activity is running it displays dynamic information about the player 
 train performance up to that moment.
-
-If the "Debrief evaluation" checkbox is unchecked, a message reminds this.
-
-   
-
-.. image:: images/dbfeval-evaluation-unchecked.png
-   :scale: 100%
-   :align: center
-
-
-In case that Debrief evaluation was checked, **Actual status: (**\ |darr| **)**\ , is displayed.
-
 
 
 .. image:: images/dbfeval-evaluation-ini.png
@@ -691,7 +678,7 @@ In case that Debrief evaluation was checked, **Actual status: (**\ |darr| **)**\
 
    \newpage
 
-Clicking **Actual status: (**\ |darr| **)**\  expanded real-time display appears.
+Clicking **Actual status: (**\ |darr| **)**\  shows an expanded real-time display.
 
 
 .. image:: images/dbfeval-evaluation-expanded.png
@@ -702,8 +689,6 @@ Clicking **Actual status: (**\ |darr| **)**\  expanded real-time display appears
 
 Clicking **Actual status: (**\ |uarr| **)**\  collapses all items.
 
-Once the activity has ended, the report file is created and a new window displays it.
-
 .. image:: images/dbfeval-evaluation-ended.png
    :scale: 100%
    :align: center
@@ -713,6 +698,9 @@ Once the activity has ended, the report file is created and a new window display
 
    \newpage
 
+Once the activity has ended, as soon as the player views the Evaluation tab, a report file is 
+created and shown in an editor window as follows.
+
 This report is made up of several sections.
 
 .. image:: images/dbfeval-report.png
@@ -721,15 +709,12 @@ This report is made up of several sections.
 
 .. raw:: latex
 
-    \newpage
+The report file OpenRailsEvaluation.txt is saved alongside the log file OpenRailsLog.txt and
+the default location for this is the Windows Desktop.
 
-Activity saves (F2) will save also the evaluation data, if the "Debrief evaluation" 
-checkbox was checked.
-In such case the activity saves will have the "Eval" checkbox checked in the resume window.
+The Save Game (F2) command also copies any evaluation report alongside the save files so it can be
+kept and reviewed. This copy is deleted when the other files for that save are deleted.
 
-.. image:: images/dbfeval-resume.png
-   :scale: 100%
-   :align: center
 
 .. _driving-hud:
 

Source/Documentation/Manual/options.rst

@@ -514,15 +514,6 @@ pressing ``<F12>`` a file with the name dump.csv is generated in the
 configured Open Rails logging folder (placed on the Desktop by default).
 This file can be used for later analysis.
 
-Evaluation Options
-==================
-
-.. image:: images/options-evaluation.png
-
-When data logging is started (see preceding paragraph), data selected in
-this panel are logged, allowing a later evaluation on how the activity was
-executed by the player.
-
 .. _options-Content:
 
 Content Options

Source/Documentation/Manual/start.rst

@@ -122,18 +122,20 @@ by that locomotive will appear.
 If you instead select a specific activity, you won't have to perform any 
 further selections.
 
-If you have selected an activity, the "Debrief evaluation" checkbox becomes 
-checkable. If it is checked, during the activity run 
-data about the activity run are stored and may be displayed in real time, and at 
-the end of the activity a report file provides a complete compendium of information about 
-the player skills as a train driver.
-Debrief evaluation is described :ref:`here <debriefeval>`.
+Activity Evaluation
+''''''''''''''''''
+
+During the activity session, data about performance is stored and may be viewed as the activity progresses.
+At the end of the activity a report file is generated which provides a summary of 
+the player's skills as a train driver.
+
+Activity evaluation is described :ref:`here <debriefeval>`.
 
 If you have selected the related Experimental Option, at runtime you can 
 switch :ref:`Autopilot mode <driving-autopilot>` on or off, which allows you
 to watch OR driving your 
 train, as if you were a trainspotter or a visitor in the cab. 
-Autopilot mode is not operating in Explore mode.
+Autopilot mode is not available in Explore mode.
 
 .. _start-timetable:
 

Source/Menu/MainForm.Designer.cs

@@ -395,13 +395,6 @@ void comboBoxActivity_SelectedIndexChanged(object sender, EventArgs e)
             ShowStartAtList();
             ShowEnvironment();
             ShowDetails();
-            //Debrief Activity Eval
-            //0 = "- Explore route -"
-            //1 = "+ Explore in Activity mode +"
-            if (comboBoxActivity.SelectedIndex < 2)
-            { checkDebriefActivityEval.Checked = false; checkDebriefActivityEval.Enabled = false; }
-            else
-            { checkDebriefActivityEval.Enabled = true; }
         }
         #endregion
 
@@ -659,10 +652,6 @@ void LoadOptions()
         {
             checkBoxWarnings.Checked = Settings.Logging;
             checkBoxWindowed.Checked = !Settings.FullScreen;
-            //Debrief activity evaluation
-            checkDebriefActivityEval.Checked = Settings.DebriefActivityEval;
-            //TO DO: Debrief TTactivity evaluation
-            //checkDebriefTTActivityEval.Checked = Settings.DebriefTTActivityEval;
             radioButtonModeActivity.Checked = Settings.IsModeActivity;
             radioButtonModeTimetable.Checked = !Settings.IsModeActivity;
 
@@ -675,10 +664,6 @@ void SaveOptions()
             Settings.Logging = checkBoxWarnings.Checked;
             Settings.FullScreen = !checkBoxWindowed.Checked;
             Settings.Multiplayer_User = textBoxMPUser.Text;
-            //Debrief activity evaluation
-            Settings.DebriefActivityEval = checkDebriefActivityEval.Checked;
-            //TO DO: Debrief TTactivity evaluation
-            //Settings.DebriefTTActivityEval = checkDebriefTTActivityEval.Checked;
             Settings.IsModeActivity = radioButtonModeActivity.Checked;
 
             var mpHost = textBoxMPHost.Text.Split(':');