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

randomTrips.py: patch comment header of output files processed by duarouter #10973

Closed
namdre opened this issue Jun 17, 2022 · 6 comments
Closed
Assignees
Milestone

Comments

@namdre
Copy link
Contributor

namdre commented Jun 17, 2022

when running randomTrips with option --validate the trips are processed by duarouter and the fact that they were created by randomTrips.py is no longer retrievable from their header. (The same applies when generating route output with option -r)

@behrisch
Copy link
Contributor

maybe having both headers is even more useful

@behrisch
Copy link
Contributor

@bcoueraud87 for you

@behrisch behrisch added this to the 1.15.0 milestone Jun 21, 2022
@bcoueraud87
Copy link
Contributor

@behrisch yes, please assign me

@namdre
Copy link
Contributor Author

namdre commented Jun 29, 2022

@bcoueraud87 only the part "The same applies when generating route output with option -r" is still missing.

@namdre
Copy link
Contributor Author

namdre commented Jun 29, 2022

@behrisch Do you see a feasible way of testing this with texttest? I mean, can we overwrite the run_dependent_text setting for a single test? We still need to filter out the timestamp but could keep the rest of the header comment since it's all relative paths.

@behrisch
Copy link
Contributor

No, I don't think so but just try it out

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

No branches or pull requests

3 participants