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

GeNet reads comma separated attributes into sets #125

Open

KasiaKoz opened this issue Jun 27, 2022 · 1 comment

Labels

Collaborator

KasiaKoz commented Jun 27, 2022

https://github.com/arup-group/genet/blob/master/genet/inputs_handler/matsim_reader.py#L114

d['text'] = set(elem.text.split(','))

This can be a problem for:

things that need to preserve order
items that we need to retain multiples of

Right now there are no documented cases of this needing to be supported but we record this issue here.

KasiaKoz added the enhancement label

KasiaKoz mentioned this issue

Magical java typing for additional attributes #124

Merged

Collaborator Author

KasiaKoz commented Jun 28, 2022

https://arupdigital.atlassian.net/browse/LAB-1750

KasiaKoz mentioned this issue

Fix reading java array type #216

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment