-
Notifications
You must be signed in to change notification settings - Fork 449
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clarify NoiseAnalysis #3519
Comments
This was referenced Nov 7, 2024
From talking to @GregorRyb, I understood that
I now added #3557, which
I hope and think that this issue can be closed as a consequence!? Let's discuss :) |
tschlenther
moved this from Todo (high priority)
to To Be Reported
in MATSIM advanced weekly hackathon
Nov 11, 2024
tschlenther
closed this as completed
by moving to To Be Reported
in
MATSIM advanced weekly hackathon
Nov 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The class
NoiseAnalysis
, which is used from the noise analysis branch of the application contrib, probably does not exclude bicycles. This needs to be understood and clarified, and then the most typical use cases either need to be included by default, or made configurable.It seems that all modes that are not explicitly excluded are classified as "pkw". I am wondering a bit if we should maybe proceed as in emissions, where the emissions vehicle type needs to be in the vehicle attributes.
This should then possibly be set in the scenario runscript (so that allVehicles already contains the correct information). (The same would then hold for particulate emissions.)
Related:
#3520
To check:
#3328 #3315 #3495
The text was updated successfully, but these errors were encountered: