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

[BUG] map in app loads extremely slow #5981

Closed
1 of 3 tasks
RenautMestdagh opened this issue Dec 26, 2023 · 8 comments
Closed
1 of 3 tasks

[BUG] map in app loads extremely slow #5981

RenautMestdagh opened this issue Dec 26, 2023 · 8 comments
Assignees

Comments

@RenautMestdagh
Copy link
Contributor

The bug

When using the map in the mobile app,it needs al very long time to load

Screenrecorder-2023-12-26-05-01-25-186.mp4

The OS that Immich Server is running on

Demo server

Version of Immich Server

V1.91.4

Version of Immich Mobile App

V1.91.4

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

/

Your .env content

/

Reproduction steps

1. In mobile app, go to search -> Your map

Additional information

No response

@alextran1502
Copy link
Contributor

I guess this is the limitation of the tile server and also based on the location where you are using the app. I don't think there is anything that we can do about this

@alextran1502 alextran1502 closed this as not planned Won't fix, can't repro, duplicate, stale Dec 27, 2023
@bo0tzz
Copy link
Member

bo0tzz commented Dec 27, 2023

@alextran1502 I think this issue is worth investigating more in depth. The tile server performs perfectly well on the web client, and I also have this same issue in the app (while pulling up the map view in my mobile browser works just fine).

@bo0tzz bo0tzz reopened this Dec 27, 2023
@aisbergde
Copy link

aisbergde commented Dec 28, 2023

I confirm that loading the map takes very long, depending on the count of items. In the browser the map loads much faster, than in the app. And I like the map in the browser much more, which shows numbers. The heat map is "uggly". And because I am color handicapped, the used colors are not well selected. Maybe to create this heat map from ten thousands items is not optimal. Why we can't get the same nice map as in the web browser?

@shenlong-tanwen shenlong-tanwen self-assigned this Dec 28, 2023
@shenlong-tanwen
Copy link
Member

The heat map is "uggly". And because I am color handicapped, the used colors are not well selected. Maybe to create this heat map from ten thousands items is not optimal. Why we can't get the same nice map as in the web browser?

Apologies for the inconvenience, the colors certainly can be updated for better accessibility. Can you open a new issue to update the colors of the heat-map? Can you also create a FR to provide an option to show clusters instead of the heat-map in the mobile app? We can discuss if we can replace the heat-map with the clusters or provide clusters as an additional option in the discussion thread. Let's keep this issue scoped to the performance of rendering the map in the mobile app which is certainly not related to the heat map generation ':)

@shenlong-tanwen
Copy link
Member

@alextran1502 I think this issue is worth investigating more in depth. The tile server performs perfectly well on the web client, and I also have this same issue in the app (while pulling up the map view in my mobile browser works just fine).

I'm thinking of getting together a POC in the near future with maplibre-gl flutter library instead of the currently used flutter_map and I hope it performs better than the current implementation. Let's keep this issue opened until the performance issue is addressed.

@aisbergde
Copy link

For me it looks like the required time to render the map is somehow proportional to the items count. That's why my idea was, that it could be related to the heat map: If the map rendering is based on getting all coordinates for all items first, then it takes some time. If the map would be generated based on clustered blocks, much less data would be need. The work could be done in the backend on the server. Depending on implemtation, the database engine could do the work and just deliver the result for clusters, which should be faster.

I don't know, how it is implemented. I only think about, how I would do. And because my main business is Business Intelligence (databases, reporting, ...), I normally try to solve performance issues in the backend.

But if it would be a backend issue, then the issue should happen also in the web browser. But there it is fast. That's why I think, it's a frontend issue, or it is related to a different approach on mobile and in the map, getting different data (detailed per item versus per cluster). And because the visible difference is heat map versus cluster, my idea was, that the heat map could be the issue. But I could be wrong. Just wanted to explain, why I wrote what I wrote.

@cedrik72
Copy link

I have the same problem, well almost the same...
The map doesnt swow photos on mobile device, and finally app crash, on the web page it's working perfectly ...

I think there is a bug on the map of the app.
(allupdated, samsung gaaxyS23Ultre mobile, server running on Docker, on local LAN network)

@bo0tzz
Copy link
Member

bo0tzz commented Apr 28, 2024

#4174

@bo0tzz bo0tzz closed this as not planned Won't fix, can't repro, duplicate, stale Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants