Skip to content
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

Update nctcog hardcoded constants #17

Draft
wants to merge 20 commits into
base: master
Choose a base branch
from
Draft

Conversation

ribsthakkar
Copy link
Member

No description provided.

@ribsthakkar ribsthakkar marked this pull request as draft September 17, 2020 20:24
finalizeNetwork blanks the class costs to initialize them.  But the NCTCOG network generates class costs before finalizeNetwork is called, so we need to preserve them.  This is a bit of an ugly hack... we should clean up the order of generating class costs for both NCTCOG and TNTP after the validation is done.
If the conic parameter is missing in the network file ("--") replace it with 12
Old toll factors were for PM peak, not AM
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants