tool for evaluation changes in route lane heuristics#1709
Merged
Framstag merged 7 commits intoFramstag:masterfrom Apr 30, 2026
Merged
tool for evaluation changes in route lane heuristics#1709Framstag merged 7 commits intoFramstag:masterfrom
Framstag merged 7 commits intoFramstag:masterfrom
Conversation
Tooling to evaluate how SuggestedLanesPostprocessor changes affect routing: generate random routes as JSON, re-compute after changes, and compare lane suggestions with optional map image rendering.
…o lane-evaluation
Collaborator
Author
|
I am not sure with the copy of nlohmann/json library in the test directory. It is the second place where we are using this simple, but useful library (it is in libosmscout-client). Should not we require it as external dependency? |
Owner
|
Yes usage of KI is OK, if the code is checked to be correct. I also have no problem with checking in KI supporting artifacts as long:
If this is always possible, has to be discussed. I also do experiments with KI usage. I currently favor spec-driven approaches, but have not yet decided for a specific tool. Currently, all seem to be a little bit bloated and require high token usage, especially for brownfield development. |
This file contains hidden or 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
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.
Tooling to evaluate how SuggestedLanesPostprocessor changes affect
routing: generate random routes as JSON, re-compute after changes,
and compare lane suggestions with optional map image rendering.
Disclaimer: These tools are for manual testing. Completely vibe-coded after an hour of brainstorming with LLM. It provided me valuable output when testing #1705 , so it works somehow. But I do not review the code properly yet, so it is a "draft" from my point of view.
@Framstag are you ok with vibe-coded changes after some level of review?