Skip to content

Add performance test for switzerland - #4576

Merged
leonardehrenfried merged 3 commits into
opentripplanner:dev-2.xfrom
slvlirnoff:switzerland-speed-test
Nov 11, 2022
Merged

Add performance test for switzerland#4576
leonardehrenfried merged 3 commits into
opentripplanner:dev-2.xfrom
slvlirnoff:switzerland-speed-test

Conversation

@slvlirnoff

Copy link
Copy Markdown
Member

Summary

Add switzerland performance test, it contains two categories:

  • stop to stop searches
  • address to address searches

The stops are sourced from the gtfs file and the addresses from openaddress.

@leonardehrenfried Let me know if it would be best to keep 'transit' as category, as it might make it otherwise painful to switch from one location to another, but I see that is already the case for Skanetrafiken tests. Maybe the dashboard could be improved to handle this based on the location selected?

@slvlirnoff
slvlirnoff requested a review from a team as a code owner November 5, 2022 11:12
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Base: 60.46% // Head: 60.47% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (1a2efe2) compared to base (a9aad86).
Patch coverage: 51.70% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #4576      +/-   ##
=============================================
+ Coverage      60.46%   60.47%   +0.01%     
- Complexity     12112    12122      +10     
=============================================
  Files           1574     1577       +3     
  Lines          63274    63232      -42     
  Branches        7005     6992      -13     
=============================================
- Hits           38257    38240      -17     
+ Misses         22838    22807      -31     
- Partials        2179     2185       +6     
Impacted Files Coverage Δ
...aphqlapi/datafetchers/LegacyGraphQLAgencyImpl.java 24.44% <0.00%> (+0.53%) ⬆️
...raphqlapi/datafetchers/LegacyGraphQLAlertImpl.java 12.50% <0.00%> (-0.08%) ⬇️
...graphqlapi/datafetchers/LegacyGraphQLFeedImpl.java 11.11% <0.00%> (ø)
...phqlapi/datafetchers/LegacyGraphQLPatternImpl.java 13.27% <0.00%> (ø)
...qlapi/datafetchers/LegacyGraphQLQueryTypeImpl.java 6.19% <0.00%> (+0.04%) ⬆️
...raphqlapi/datafetchers/LegacyGraphQLRouteImpl.java 17.00% <0.00%> (+0.33%) ⬆️
...graphqlapi/datafetchers/LegacyGraphQLStopImpl.java 11.06% <0.00%> (-0.09%) ⬇️
...graphqlapi/datafetchers/LegacyGraphQLTripImpl.java 13.40% <0.00%> (ø)
...transmodelapi/model/siri/et/EstimatedCallType.java 67.33% <0.00%> (+0.27%) ⬆️
...modelapi/model/siri/sx/PtSituationElementType.java 58.06% <0.00%> (ø)
... and 17 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread test/performance/switzerland/build-config.json Outdated
@vpaturet
vpaturet self-requested a review November 10, 2022 14:41
@leonardehrenfried
leonardehrenfried merged commit 53e30c6 into opentripplanner:dev-2.x Nov 11, 2022
t2gran pushed a commit that referenced this pull request Nov 11, 2022
@t2gran t2gran added this to the 2.3 milestone Feb 15, 2023
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.

7 participants