Skip to content

Add debug layer for rental entities - #6520

Merged
leonardehrenfried merged 4 commits into
opentripplanner:dev-2.xfrom
ibi-group:rental-debug
Mar 11, 2025
Merged

Add debug layer for rental entities#6520
leonardehrenfried merged 4 commits into
opentripplanner:dev-2.xfrom
ibi-group:rental-debug

Conversation

@leonardehrenfried

Copy link
Copy Markdown
Member

Summary

It adds a debug layer for displaying vehicle rental entities in the debug UI.

Screenshot From 2025-03-07 14-32-04

Unit tests

Updated.

@leonardehrenfried leonardehrenfried added the +OTP Debug UI OTP bundled client, the OTP server is normally not changed label Mar 7, 2025
@leonardehrenfried
leonardehrenfried requested a review from a team as a code owner March 7, 2025 13:38
@codecov

codecov Bot commented Mar 7, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 74.32432% with 19 lines in your changes missing coverage. Please review.

Project coverage is 70.18%. Comparing base (9d01118) to head (81f1211).
Report is 22 commits behind head on dev-2.x.

Files with missing lines Patch % Lines
...er/inspector/vector/rental/RentalLayerBuilder.java 0.00% 12 Missing ⚠️
.../vectortiles/GraphInspectorVectorTileResource.java 0.00% 4 Missing ⚠️
.../inspector/vector/rental/RentalPropertyMapper.java 88.88% 1 Missing and 1 partial ⚠️
...ntripplanner/apis/vectortiles/model/LayerType.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             dev-2.x    #6520   +/-   ##
==========================================
  Coverage      70.17%   70.18%           
- Complexity     18312    18318    +6     
==========================================
  Files           2082     2084    +2     
  Lines          77211    77278   +67     
  Branches        7831     7834    +3     
==========================================
+ Hits           54184    54234   +50     
- Misses         20257    20272   +15     
- Partials        2770     2772    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@leonardehrenfried

Copy link
Copy Markdown
Member Author

cc @fpurcell

Comment on lines +14 to +21
protected Collection<KeyValue> map(VehicleRentalPlace input) {
return List.of(
kv("class", input.getClass().getSimpleName()),
kv("id", input.getName()),
kv("network", input.getNetwork()),
kv("vehiclesAvailable", input.getVehiclesAvailable()),
kv("spacesAvailable", input.getSpacesAvailable())
);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps you could also add information about if the station/vehicle is disabled or not.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I'm sorry didn't remember we even had these disabled vehicles/spaces. I meant the isRenting/isReturning type of information. I'm not sure how often this disabled vehicle/space information is provided by the data feeds in cases where the station is not in service. We can include both information if you want.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@leonardehrenfried
leonardehrenfried merged commit 532da22 into opentripplanner:dev-2.x Mar 11, 2025
t2gran pushed a commit that referenced this pull request Mar 11, 2025
@leonardehrenfried
leonardehrenfried deleted the rental-debug branch March 11, 2025 13:47
@t2gran t2gran added this to the 2.7 milestone Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

+OTP Debug UI OTP bundled client, the OTP server is normally not changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants