Default setting for 'mutationRangeStep '

[lukknapen]

• Does the default setting

   <module name="timeAllocationMutator" >
      <!-- Mutation Range Step, default = 1 second -->
      <param name="mutationRangeStep" value="1.0" />
   </module>

make sense ? In org/matsim/core/population/algorithms/PlanMutateTimeAllocation.java
and org/matsim/core/population/algorithms/PlanMutateTimeAllocationSimplified.java the second term in

double t = time.seconds() + (int)((this.random.nextDouble() * 2.0 - 1.0) * this.mutationRange);

always equals zero for the default setting due to the 'double to int' type casting.
Hence, the default value does not induce timeMutation.

This is because (int)(someDoubleValue) uses the 'rounding to zero method';
see https://docs.oracle.com/javase/specs/jls/se17/html/jls-5.html#jls-5.1.3

• Was this default value chosen on purpose ? If so, it is useful to extend the comment for this setting in the logFile.log It now just says <!-- Mutation Range Step, default = 1 second --> which is correct but may be insufficient.