Fix/emme clarifications #156
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zptro
reviewed
Jan 4, 2023
docs/emme_verkko.md
Outdated
| attribuuteista sijoittelun jälkeen. Kaikille katu- ja tieverkon linkeille ei kuitenkaan ole saatu | ||
| tuotettua taulukko 11:n mukaista linkkityyppiä. Näiden linkkien tyypiksi on koodattu 191-195 (sekä | ||
| 291-295, 391-395, ... linkeille, joilla on bussikaista tai jotka ovat joukkoliikennekatuja, ja | ||
| jotka eivät noudata taulukkoa 11), ja niiden alkuperäiset vdf-, ul1-, ja ul2-attribuutit säilyvät |
There was a problem hiding this comment.
Vdf otetaan linkkityypin viimeisestä numerosta, joten se ei välttämättä säily sijoittelun aikana. Tässä menetelmä on kuvattu tarkemmin:
Line 298 in 7a24982
There was a problem hiding this comment.
Korjasin vähän tekstiä. Luulin, että _set_car_and_transit_vdfs() nimenomaan tulkitsisi vdf:t ja vie ne lopulta Emme-skenaarioon (publish_network)? https://github.com/HSLdevcom/helmet-model-system/blob/olusanya/Scripts/assignment/assignment_period.py#L270
There was a problem hiding this comment.
Custom-linkeille lasketaan linktypestä:
There was a problem hiding this comment.
Moi! Tässä on nyt kaksi asiaa:
- Olin kirjoittanut aiemmin, että "Näiden linkkien tyypiksi on koodattu 191-195 [...], ja niiden alkuperäiset vdf-, ul1-, ja ul2-attribuutit säilyvät sijoittelun aikana.". Tässä oli väärin se, että vdf-attribuutit säilyisivät sijoittelun aikana, jota kommentoit, joten korjasin muotoon: "Näiden linkkien tyypiksi on koodattu 191-195 [...], ja niiden alkuperäiset ul1- ja ul2-attribuutit säilyvät sijoittelun aikana. Niiden vdf-attribuutti päätellään kuten taulukossa 10 kerrotaan." Tämä lienee nyt ratkaistu?
- Toinen asia on, että halusimme alleviivata sitä, että sijoittelu ylikirjoittaa alkuperäisiä vdf-, ul1- ja ul2-attribuutteja tietyissä tapauksissa. Vaikka linkkityypeillä 191-195 vdf tulkitaan eri tavalla kuin muilla linkkityypeillä, ne edelleen kirjoitetaan verkolle alkuperäisten päälle, eikö? Eli se, mitä vdf sisältää alunperin, on samantekevää?
zptro
approved these changes
Jan 4, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #154 and #155.
I tried to underline the fact that one can not trust vdf, ul1, or ul2 attributes if they have not done an assignment first. It is only after assignment that all attributes are correct, as type overrides everything else.