From 6a30c28d370e72729199eee0d9a48741f9cc98d8 Mon Sep 17 00:00:00 2001
From: schlenther <schlenther@vsp.tu-berlin.de>
Date: Wed, 7 Aug 2024 17:24:55 +0200
Subject: [PATCH] NoiseAnalysis: set the scale factor for according to sample
 size

---
 .../analysis/noise/NoiseAnalysis.java         | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/contribs/application/src/main/java/org/matsim/application/analysis/noise/NoiseAnalysis.java b/contribs/application/src/main/java/org/matsim/application/analysis/noise/NoiseAnalysis.java
index 7cd5fea5502..6730cae8996 100644
--- a/contribs/application/src/main/java/org/matsim/application/analysis/noise/NoiseAnalysis.java
+++ b/contribs/application/src/main/java/org/matsim/application/analysis/noise/NoiseAnalysis.java
@@ -1,5 +1,7 @@
 package org.matsim.application.analysis.noise;
 
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.locationtech.jts.geom.Envelope;
 import org.matsim.api.core.v01.Coord;
 import org.matsim.api.core.v01.Scenario;
@@ -8,6 +10,7 @@
 import org.matsim.application.MATSimAppCommand;
 import org.matsim.application.options.InputOptions;
 import org.matsim.application.options.OutputOptions;
+import org.matsim.application.options.SampleOptions;
 import org.matsim.application.options.ShpOptions;
 import org.matsim.contrib.noise.NoiseConfigGroup;
 import org.matsim.contrib.noise.NoiseOfflineCalculation;
@@ -38,6 +41,8 @@
 )
 public class NoiseAnalysis implements MATSimAppCommand {
 
+	private static final Logger log = LogManager.getLogger(NoiseAnalysis.class);
+
 	@CommandLine.Mixin
 	private final InputOptions input = InputOptions.ofCommand(NoiseAnalysis.class);
 	@CommandLine.Mixin
@@ -46,6 +51,9 @@ public class NoiseAnalysis implements MATSimAppCommand {
 	@CommandLine.Mixin
 	private final ShpOptions shp = new ShpOptions();
 
+	@CommandLine.Mixin
+	private final SampleOptions sampleOptions = new SampleOptions();
+
 	@CommandLine.Option(names = "--consider-activities", split = ",", description = "Considered activities for noise calculation." +
 		" Use asterisk ('*') for acttype prefixes, if all such acts shall be considered.", defaultValue = "h,w,home*,work*")
 	private Set<String> considerActivities;
@@ -89,6 +97,16 @@ public Integer call() throws Exception {
 			noiseParameters.setNoiseBarriersFilePath(noiseBarrierFile);
 		}
 
+		if(! sampleOptions.isSet() && noiseParameters.getScaleFactor() == 1d){
+			log.warn("You didn't provide the simulation sample size via command line option --sample-size! This means, noise damages are not scaled!!!");
+		} else if (noiseParameters.getScaleFactor() == 1d){
+			if (sampleOptions.getSample() == 1d){
+				log.warn("Be aware that the noise output is not scaled. This might be unintended. If so, assure to provide the sample size via command line option --sample-size, in the SimWrapperConfigGroup," +
+					"or provide the scaleFactor (the inverse of the sample size) in the NoiseConfigGroup!!!");
+			}
+			noiseParameters.setScaleFactor(sampleOptions.getUpscaleFactor());
+		}
+
 		Scenario scenario = ScenarioUtils.loadScenario(config);
 
 		String outputFilePath = output.getPath().getParent() == null ? "." : output.getPath().getParent().toString();
@@ -120,6 +138,7 @@ private Config prepareConfig() {
 		config.facilities().setInputFile(null);
 		config.eventsManager().setNumberOfThreads(null);
 		config.eventsManager().setEstimatedNumberOfEvents(null);
+		//ts, aug '24: not sure if and why we need to set 1 thread
 		config.global().setNumberOfThreads(1);
 
 		return config;