-
Notifications
You must be signed in to change notification settings - Fork 3
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
Vehicle kms and noise calculation #269
Conversation
zptro
commented
Mar 10, 2021
•
edited
edited
- Sum vehicle kilometres to area level
- Calculate noise areas
- Print numbers of metro and train stations (for use in CBA maintenance cost calculation)
- Save car time matrices in Emme
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! I added a couple of comments regarding documentations. One other issue though... Are we still using pylint
? The linter throws 215 problems in these changed files (trailing whitespace, too long lines, missing docstrings, naming conventions...). We could also do a separate PR for cleaning those here and in other scripts.
The question is rather "Are we still not using A separate PR for fixing all the bad coding is a good idea. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes! Looks good to me!
@johpiip, maybe this is a better way of representing the step-wise function? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple optional changes. Nice refactoring!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!