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.

Sign up for GitHub

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

Jump to bottom

Is it enough to edit only types in the network? #154

Closed
johpiip opened this issue Jan 3, 2023 · 2 comments · Fixed by #156
Closed

Is it enough to edit only types in the network? #154

johpiip opened this issue Jan 3, 2023 · 2 comments · Fixed by #156
Assignees
@johpiip @eevavesaoja

Comments

@johpiip
Copy link
Contributor

johpiip commented Jan 3, 2023 •
edited
Loading

@eevavesaoja :n jatkokysymys #145 :ssä:

"Sijoitteluskripti muuttaa linkien funktio-, nopeus- ja kapasiteettiattribuutit (vdf, ul1, ul2) type-attribuutin perusteella. Kaikille katu- ja tieverkon linkeille ei ole saatu tuotettua taulukko 11:n mukaista linkkityyppiä. "
Tarkoitetaanko tällä ohjetta, että ”Koodaa yleisesti ottaen pelkkä type”?

Originally posted by @eevavesaoja in #145 (review)
@zptro
Copy link
Collaborator

zptro commented Jan 3, 2023

Tässä referenssi ko. kohtaan:

helmet-ui/docs/emme_verkko.md

Line 244 in f5b0363

Sijoitteluskripti muuttaa linkien funktio-, nopeus- ja kapasiteettiattribuutit (vdf, ul1, ul2) type-attribuutin perusteella.

Tarkoitetaanko tällä ohjetta, että ”Koodaa yleisesti ottaen pelkkä type”?

Kyllä tarkoittaa.

@johpiip
Copy link
Contributor Author

johpiip commented Jan 3, 2023

Kiitos @zptro, mutta me selvitettiin sama jo tässä päässä. 🙂 Nostin tämän tänne sitä varten, että tulee kirjattua ja keskusteltua.

johpiip added a commit that referenced this issue Jan 3, 2023
@johpiip
Fix #154
f9d77b1
@johpiip johpiip mentioned this issue Jan 3, 2023
Merged
johpiip added a commit that referenced this issue Jan 4, 2023
@johpiip
Fix/emme clarifications (#156)
0518d6c 
* Fix #154

* Fix #155

* Add changelog

* Fix error in vdfs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johpiip johpiip

@eevavesaoja eevavesaoja

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix/emme clarifications HSLdevcom/helmet-ui
3 participants
@zptro @johpiip @eevavesaoja