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;