-
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
Generalize necessary changes for MTC to work for MetCouncil and other agencies into the future #236
Conversation
1. Scalar values were different 2. MTC needed additional variables copied to managed lanes 3. Different foreign key for shapes 4. Scenario initiation from a Scenario instance OR a dictionary representing one Co-authored-by: David Ory <[email protected]>
- moved parameters from client.py to modules - added params to class instances - parameterized methods for non instance-level vars TODO: update all notebooks and usage.
Fixes #229 by making methods parameterizable and storing most client settings in RoadwayNetwork and TransitNetwork instance-level with good defaults rather than class-level variables. Also: - Added notebook testing - Some other clean up
- fixed typo bugs
- Allow new columns to be added wholesale - Also fixes #257
- output links, nodes, shapes for managed lane network - externalize `has_managed_lanes()`
+ some PEP8
Hello - is it possible to prioritize this PR? I'm concerned that the distance between |
Hi again, @DavidOry , @i-am-sijia and @e-lo -- Just pinging on this again; what's the plan/timeline? Thank you! |
Yes. Discussed this morning with @e-lo. What's your best guess at a timeline @e-lo? |
Update Travis from using "travis" to "CI"
- exclude validation status on write by default. - add an explicit write-order by default
CLosing this as having been superceeded by work in |
Closes Issue #229