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

optimizes road objects creation by re-using the parsing that was already done #6375

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

LukasPaczos
Copy link
Member

Description

Skips the manual parsing work and re-uses the results that were already done during route generation.

@LukasPaczos LukasPaczos added needs backporting Requires cherry-picking to a currently running release branch skip changelog Should not be added into version changelog. improvement Improvement for an existing feature. labels Sep 23, 2022
@LukasPaczos LukasPaczos requested a review from a team as a code owner September 23, 2022 09:35
@LukasPaczos LukasPaczos force-pushed the lp-optimize-road-objects-parsing branch from 4341362 to 38c2857 Compare September 23, 2022 10:19
@LukasPaczos LukasPaczos enabled auto-merge (rebase) September 23, 2022 10:49
@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #6375 (38c2857) into main (0b00489) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #6375   +/-   ##
=========================================
  Coverage     68.89%   68.89%           
  Complexity     4552     4552           
=========================================
  Files           686      686           
  Lines         27211    27209    -2     
  Branches       3179     3179           
=========================================
- Hits          18747    18746    -1     
  Misses         7240     7240           
+ Partials       1224     1223    -1     
Impacted Files Coverage Δ
.../navigation/core/trip/session/MapboxTripSession.kt 86.56% <100.00%> (+0.21%) ⬆️

@LukasPaczos LukasPaczos merged commit 4f2722d into main Sep 23, 2022
@LukasPaczos LukasPaczos deleted the lp-optimize-road-objects-parsing branch September 23, 2022 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement for an existing feature. needs backporting Requires cherry-picking to a currently running release branch skip changelog Should not be added into version changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants