Skip to content

Alter event_reader definition - #33

Merged
theresamariaM merged 4 commits into
matsim-vsp:masterfrom
theresamariaM:master
Sep 22, 2025
Merged

Alter event_reader definition#33
theresamariaM merged 4 commits into
matsim-vsp:masterfrom
theresamariaM:master

Conversation

@theresamariaM

Copy link
Copy Markdown
Contributor

The event_reader definition was altered to remove leading and trailing white spaces from the event types if they are a string.

@theresamariaM
theresamariaM merged commit 4b2f4e2 into matsim-vsp:master Sep 22, 2025
1 check passed
@theresamariaM
theresamariaM requested a review from rakow September 22, 2025 11:57
@theresamariaM

Copy link
Copy Markdown
Contributor Author

Previously the event_reader definition did not allow filtering multiple event types if an event had a leading or trailing whitespace after a comma. Example: types = "link leave, link enter" would have resulted in only reading the link leave event. While entering types = "link leave,link enter" would have resulted in reading both events. Therefore, the definition was altered to remove leading and trailing whitespaces.

@rakow

rakow commented Sep 22, 2025

Copy link
Copy Markdown
Collaborator

PR was approved after discussing in person.

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