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

Fix #8323 Add the name of the street driving on #8347

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

alrega
Copy link

@alrega alrega commented Jan 28, 2020

I've modified source code in order to implement improvement #8323
I've tested changes by using Android Emulator, but not in the street using instant hardware.

Could you share APK file built in your CI with this change? I'd like to make system testing. I have aarch64 (arm64-v8a) CPU. Alternatively, please let me know Gradle task name to build APK locally.

Thank you in advance.

Fix bug in improper visibility evaluation of current street name
@alrega
Copy link
Author

alrega commented Feb 7, 2020

@vshcherb could you review the changes and merge/rebase them?

Improve composing of current street name.
@alrega alrega requested a review from max-klaus February 9, 2020 08:59
@vshcherb vshcherb added this to the 3.8 milestone Feb 10, 2020
@vshcherb
Copy link
Member

We're not sure whether it fits all combinations or not, so we need more time to review it & test it.
There is no immediate plan to act on this request.

@vshcherb vshcherb modified the milestones: 3.8, Future milestone Mar 31, 2020
@vshcherb vshcherb added the Nice to Have Should be fixed but there is no priority or no possibility to fix it within current horizon planning label Dec 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Nice to Have Should be fixed but there is no priority or no possibility to fix it within current horizon planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants