-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Nearby: Some pins disappear after switching apps #5482
Comments
I think this issue is related to that configuration change issue that I am currently working on in my other PRs , can you assign this to me @nicolas-raoul |
Did you find out where is the problem? If not then please find it first :) |
I think it is fine to get assigned before starting to investigate. :-) |
Yes, you are right. But what I think is, that we should first reproduce the issue and then have a scenario ahead of how a particular issue will be solved. |
Yes, being able to reproduce can probably be considered a minimum. Different projects have different rules/habits though. 🙂 |
Well yeah 🙂 |
@nicolas-raoul I examined the issue well and I was able to reproduce it as you can see below WhatsApp.Video.2024-02-04.at.12.07.03.PM.mp4on deep examination, I found out that this issue is not of some pins disappearing but rather extra pins loading when the view is created, which is because of a duplicate call for Due to this repeated call, an extra number of pins are loaded (which could have been otherwise loaded when recentering the map and searching the area), but when you switch apps only |
Working fine now. |
... even though I didn't change location (using FakeGPS).
Observed on main branch at 96b2608, but might have started earlier.
git bisect
will most probably help.screen-20240127-223908.mp4
The text was updated successfully, but these errors were encountered: