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

Potree v2 remote #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Pourfex
Copy link

@Pourfex Pourfex commented Aug 13, 2024

Remote URL

We would like to use the PotreeV2 format for remote url, mainly because we might have to handle several Go of data.

Remote/local URL handling with v1/v2 version

For this we had to transform the URL given to the Cloud loader :

Potree1 :
Clouds/Lion -> Clouds/Lion/cloud.js
https://remoteurl.com/Lion -> https://remoteurl.com/Lion/cloud.js

Potree2 :
Clouds/Lion2 -> Clouds/Lion2/metadata.json
https://remoteurl.com/Lion2 -> https://remoteurl.com/Lion2/metadata.json

We did this so codebase can handle in a easier way the remote/local and v1/v2 cases.

Range request

We have to do range request to get the part of the remote file we want, so we're using range request as Potree is doing.

@Pourfex Pourfex force-pushed the f/PotreeV2_remote branch from 4658f41 to af14421 Compare August 13, 2024 15:23
@Pourfex Pourfex force-pushed the f/PotreeV2_remote branch from af14421 to 00e8b6b Compare August 13, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant