WIP: Tile3D getCoordinatesByIndex - #2498
Closed
radoslavirha wants to merge 1 commit into
Closed
Conversation
Collaborator
|
Thanks for putting this together, and apologies that it did not receive a timely review. The use case is valid, but I don't think this implementation can be merged as written:
The branch is now several years behind master, has conflicts, and the If this capability is revisited, a fresh PR should probably start with a generic helper that transforms a supplied tile-local position to Cartesian or cartographic coordinates, with an optional point-cloud-specific indexed accessor layered on top. For exact WGS84 output, applying |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
regarding to my previous discussion #2425 (comment) I prepared something which I can improve with your guidance.
My final target is to create cursor similar to this.
On meshes it should be simple, maybe another method can return orientation of mesh under cursor + cursor coords and I can render oriented circle under cursor.
On point clouds it's more difficult I think, so I started with picking multiple points around cursor and I want to create plane from those (3?) points, calculate orientation,... So this is the use case where it's useful for me. But it could be usable generally too.
Thanks