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

Progressive load of objects only if they are in the current viewport #10

Open
jscastro76 opened this issue Jul 5, 2020 · 2 comments
Open
Labels

Comments

@jscastro76
Copy link
Owner

jscastro76 commented Jul 5, 2020

Check if a GeoJSON source is present in mapbox viewport
Mapbox queryRenderedFeatures

@jscastro76 jscastro76 self-assigned this Jul 6, 2020
@jscastro76 jscastro76 added this to the v2.0.4 milestone Jul 7, 2020
@jscastro76 jscastro76 modified the milestones: v2.0.4, v2.0.5. Jul 25, 2020
@joedjc
Copy link

joedjc commented Jul 29, 2020

@jscastro76 Firstly it's great to see this library being taken further with many useful features.

As a thought on this - we've been wondering how we could improve the performance of lots of 3D elements using LOD, meshes etc. I was wondering if you'd had any thoughts on the use of 3D tiles, which could be streamed from a server?

I saw some examples here:
https://github.com/Geodan/mapbox-3dtiles
https://github.com/Geodan/pg2b3dm

Which I thought were interesting. Maybe not appropriate for threebox and slightly different use case to using GeoJSON

@jscastro76
Copy link
Owner Author

Hi @joedjc,
Sorry for this delayed reply, I've been buried under a ton of work this week.
Regarding your questions, I'm definitely interested in LOD because of the potential performance improvement, I need that feature as a priority. I opened an issue #9 on that topic.
On the other side, I'm not interested (yet) on any 3D tiling like those ones from Geodan or this one from the original creator of this repo, or this one from Mapbox. I read in an issue that I opened to Mapbox that they are already working on 3D tiling as they discussed around the use of term altitude or z or elevation or offset... it was West Langley so I guess this will come directly from Mapbox.

@jscastro76 jscastro76 removed this from the v2.0.5. milestone Aug 12, 2020
@jscastro76 jscastro76 removed their assignment Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants