Repository navigation
gui: add grt heatmap to save_image -web - #12045
mikesinouye wants to merge 4 commits into
Conversation
Signed-off-by: Mike Inouye <mikeinouye@google.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds support for a global routing congestion heatmap overlay (routing_congestion) to the web tile generator. The changes include updating the TileVisibility configuration, implementing the drawRoutingCongestionLayer method to render the heatmap, integrating the new layer into the pseudo-layer definitions, and updating the documentation and unit tests. There are no review comments to assess, and the changes appear correct and well-tested, so I have no feedback to provide.
Signed-off-by: Mike Inouye <mikeinouye@google.com>
|
@Logikable would you be able to take a look when you get a chance? Just a minor nice to have for the route flow bringup internally |
Signed-off-by: Mike Inouye <mikeinouye@google.com>
|
I'm confused, there is already a routing congestion heatmap |
That's the for the GUI, which is already hooked up, this is for the renamed the ambiguous PR name to make that more clear. |
mikesinouye
changed the title
|
Hi @maliberty , and feedback to provide? |
|
|
…ROAD into pr12045-merge Signed-off-by: Matt Liberty <mliberty@precisioninno.com> # Conflicts: # src/web/test/cpp/TestSaveImage.cpp
Head branch was pushed to by a user without write access
|
@maliberty sorry to bump again, CI passed this time, auto merge got disabled |
Summary
Plumb existing global route routing congestion heatmap support from GUI to
save_image -web.Type of Change
Impact
New heatmap option.
Verification
./etc/Build.sh).