Index: Riskeer/Common/src/Riskeer.Common.Service/DesignWaterLevelCalculationActivity.cs =================================================================== diff -u -ra9dc04dde69e1d7b8ebc2207dc45644d3c06a4bc -r11e5b00b0252e6bf58813a42ea275c404f5e3c95 --- Riskeer/Common/src/Riskeer.Common.Service/DesignWaterLevelCalculationActivity.cs (.../DesignWaterLevelCalculationActivity.cs) (revision a9dc04dde69e1d7b8ebc2207dc45644d3c06a4bc) +++ Riskeer/Common/src/Riskeer.Common.Service/DesignWaterLevelCalculationActivity.cs (.../DesignWaterLevelCalculationActivity.cs) (revision 11e5b00b0252e6bf58813a42ea275c404f5e3c95) @@ -30,7 +30,7 @@ /// internal class DesignWaterLevelCalculationActivity : CalculatableActivity { - private readonly double norm; + private readonly double targetProbability; private readonly ICalculationMessageProvider messageProvider; private readonly DesignWaterLevelCalculationService calculationService; private readonly HydraulicBoundaryLocationCalculation hydraulicBoundaryLocationCalculation; @@ -42,15 +42,15 @@ /// The hydraulic boundary location calculation to perform. /// The with the /// hydraulic boundary calculation settings. - /// The norm to use during the calculation. + /// The target probability to use during the calculation. /// The calculation identifier to use in all messages. /// Preprocessing is disabled when the preprocessor directory equals . /// Thrown when or /// is null. /// Thrown when is null or empty. public DesignWaterLevelCalculationActivity(HydraulicBoundaryLocationCalculation hydraulicBoundaryLocationCalculation, HydraulicBoundaryCalculationSettings calculationSettings, - double norm, + double targetProbability, string calculationIdentifier) : base(hydraulicBoundaryLocationCalculation) { @@ -63,7 +63,7 @@ this.hydraulicBoundaryLocationCalculation = hydraulicBoundaryLocationCalculation; this.calculationSettings = calculationSettings; - this.norm = norm; + this.targetProbability = targetProbability; calculationService = new DesignWaterLevelCalculationService(); @@ -72,14 +72,14 @@ protected override bool Validate() { - return calculationService.Validate(calculationSettings, norm); + return calculationService.Validate(calculationSettings, targetProbability); } protected override void PerformCalculation() { calculationService.Calculate(hydraulicBoundaryLocationCalculation, calculationSettings, - norm, + targetProbability, messageProvider); } Index: Riskeer/Common/test/Riskeer.Common.Service.Test/DesignWaterLevelCalculationActivityTest.cs =================================================================== diff -u -r6a8b1b5c3f3d8ff1115e39401bbe7aff42a1a5f7 -r11e5b00b0252e6bf58813a42ea275c404f5e3c95 --- Riskeer/Common/test/Riskeer.Common.Service.Test/DesignWaterLevelCalculationActivityTest.cs (.../DesignWaterLevelCalculationActivityTest.cs) (revision 6a8b1b5c3f3d8ff1115e39401bbe7aff42a1a5f7) +++ Riskeer/Common/test/Riskeer.Common.Service.Test/DesignWaterLevelCalculationActivityTest.cs (.../DesignWaterLevelCalculationActivityTest.cs) (revision 11e5b00b0252e6bf58813a42ea275c404f5e3c95) @@ -43,8 +43,6 @@ [TestFixture] public class DesignWaterLevelCalculationActivityTest { - private const double validNorm = 0.005; - private static readonly string testDataPath = TestHelper.GetTestDataPath(TestDataPath.Riskeer.Integration.Service, "HydraRingCalculation"); private static readonly string validHydraulicBoundaryDatabaseFilePath = Path.Combine(testDataPath, "HRD dutch coast south.sqlite"); private static readonly string validHlcdFilePath = Path.Combine(testDataPath, "Hlcd.sqlite"); @@ -70,15 +68,11 @@ [Test] public void Constructor_CalculationSettingsNull_ThrowsArgumentNullException() { - // Setup - const string locationName = "locationName"; - const string calculationIdentifier = "1/200"; - // Call - void Call() => new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation(locationName)), + void Call() => new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation("locationName")), null, - validNorm, - calculationIdentifier); + 0.01, + "1/100"); // Assert var exception = Assert.Throws(Call); @@ -90,12 +84,12 @@ { // Setup const string locationName = "locationName"; - const string calculationIdentifier = "1/200"; + const string calculationIdentifier = "1/100"; // Call var activity = new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation(locationName)), CreateCalculationSettings(), - validNorm, + 0.01, calculationIdentifier); // Assert @@ -111,15 +105,15 @@ // Setup string invalidFilePath = Path.Combine(testDataPath, "notexisting.sqlite"); const string locationName = "locationName"; - const string calculationIdentifier = "1/200"; + const string calculationIdentifier = "1/100"; var settings = new HydraulicBoundaryCalculationSettings(invalidFilePath, validHlcdFilePath, false, string.Empty); var activity = new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation(locationName)), settings, - validNorm, + 0.01, calculationIdentifier); // Call @@ -144,15 +138,15 @@ // Setup const string invalidPreprocessorDirectory = "NonExistingPreprocessorDirectory"; const string locationName = "locationName"; - const string calculationIdentifier = "1/200"; + const string calculationIdentifier = "1/100"; var settings = new HydraulicBoundaryCalculationSettings(validHydraulicBoundaryDatabaseFilePath, validHlcdFilePath, false, invalidPreprocessorDirectory); var activity = new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation(locationName)), settings, - validNorm, + 0.01, calculationIdentifier); // Call @@ -172,7 +166,7 @@ } [Test] - public void Run_InvalidNorm_PerformValidationAndLogStartAndEndAndError() + public void Run_InvalidTargetProbability_PerformValidationAndLogStartAndEndAndError() { // Setup const string locationName = "locationName"; @@ -206,8 +200,8 @@ { // Setup const string locationName = "locationName"; - const string calculationIdentifier = "1/30"; - const double norm = 1.0 / 30; + const string calculationIdentifier = "1/100"; + const double targetProbability = 0.01; var calculator = new TestDesignWaterLevelCalculator { @@ -235,7 +229,7 @@ var activity = new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(hydraulicBoundaryLocation), calculationSettings, - norm, + targetProbability, calculationIdentifier); using (new HydraRingCalculatorFactoryConfig(calculatorFactory)) @@ -246,7 +240,7 @@ // Assert AssessmentLevelCalculationInput designWaterLevelCalculationInput = calculator.ReceivedInputs.Single(); Assert.AreEqual(hydraulicBoundaryLocation.Id, designWaterLevelCalculationInput.HydraulicBoundaryLocationId); - Assert.AreEqual(StatisticsConverter.ProbabilityToReliability(norm), designWaterLevelCalculationInput.Beta); + Assert.AreEqual(StatisticsConverter.ProbabilityToReliability(targetProbability), designWaterLevelCalculationInput.Beta); } Assert.AreEqual(ActivityState.Executed, activity.State); @@ -258,8 +252,8 @@ { // Setup const string locationName = "locationName"; - const string calculationIdentifier = "1/30"; - const double norm = 1.0 / 30; + const string calculationIdentifier = "1/100"; + const double targetProbability = 0.01; var calculator = new TestDesignWaterLevelCalculator { @@ -279,7 +273,7 @@ var activity = new DesignWaterLevelCalculationActivity(new HydraulicBoundaryLocationCalculation(hydraulicBoundaryLocation), calculationSettings, - norm, + targetProbability, calculationIdentifier); using (new HydraRingCalculatorFactoryConfig(calculatorFactory)) @@ -310,7 +304,7 @@ public void Run_ValidCalculation_SetsDesignWaterLevelAndConvergence(HydraulicBoundaryLocationCalculation hydraulicBoundaryLocationCalculation) { // Setup - const double norm = 1.0 / 30; + const double targetProbability = 0.01; const double expectedDesignWaterLevel = 3.5; var calculator = new TestDesignWaterLevelCalculator @@ -333,8 +327,8 @@ var activity = new DesignWaterLevelCalculationActivity(hydraulicBoundaryLocationCalculation, CreateCalculationSettings(), - norm, - "1/30"); + targetProbability, + "1/100"); using (new HydraRingCalculatorFactoryConfig(calculatorFactory)) { @@ -359,7 +353,7 @@ { // Setup const string locationName = "locationName"; - const string calculationIdentifier = "1/200"; + const string calculationIdentifier = "1/100"; var calculator = new TestDesignWaterLevelCalculator { @@ -386,7 +380,7 @@ var activity = new DesignWaterLevelCalculationActivity(hydraulicBoundaryLocationCalculation, CreateCalculationSettings(), - validNorm, + 0.01, calculationIdentifier); using (new HydraRingCalculatorFactoryConfig(calculatorFactory)) @@ -426,7 +420,7 @@ { // Setup const string locationName = "locationName"; - const string calculationIdentifier = "1/300"; + const string calculationIdentifier = "1/100"; var calculator = new TestDesignWaterLevelCalculator { @@ -450,10 +444,9 @@ Output = new TestHydraulicBoundaryLocationCalculationOutput(double.NaN, CalculationConvergence.CalculatedConverged) }; - const double norm = 1.0 / 300; var activity = new DesignWaterLevelCalculationActivity(hydraulicBoundaryLocationCalculation, CreateCalculationSettings(), - norm, + 0.01, calculationIdentifier); using (new HydraRingCalculatorFactoryConfig(calculatorFactory)) @@ -482,7 +475,7 @@ public void Finish_ActivityWithSpecificState_NotifyHydraulicBoundaryLocationCalculation(ActivityState state) { // Setup - const string calculationIdentifier = "1/200"; + const string calculationIdentifier = "1/100"; var hydraulicBoundaryLocationCalculation = new HydraulicBoundaryLocationCalculation(new TestHydraulicBoundaryLocation()); @@ -494,7 +487,7 @@ var activity = new TestDesignWaterLevelCalculationActivity(hydraulicBoundaryLocationCalculation, CreateCalculationSettings(), - validNorm, + 0.01, calculationIdentifier, state); @@ -522,12 +515,12 @@ { public TestDesignWaterLevelCalculationActivity(HydraulicBoundaryLocationCalculation hydraulicBoundaryLocationCalculation, HydraulicBoundaryCalculationSettings calculationSettings, - double norm, + double targetProbability, string calculationIdentifier, ActivityState state) : base(hydraulicBoundaryLocationCalculation, calculationSettings, - norm, + targetProbability, calculationIdentifier) { State = state;