Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Project point onto plane #6092

Merged
merged 4 commits into from
Jan 5, 2018
Merged

Project point onto plane #6092

merged 4 commits into from
Jan 5, 2018

Conversation

hpinkos
Copy link
Contributor

@hpinkos hpinkos commented Jan 5, 2018

Adds a function for projecting a Cartesian3 point onto a plane.
I've been using this function a lot lately, so I thought it would be helpful to have in core Cesium.

@cesium-concierge
Copy link

Signed CLA is on file.

@hpinkos, thanks for the pull request! Maintainers, we have a signed CLA from @hpinkos, so you can review this at any time.


I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

@hpinkos hpinkos requested a review from bagnell January 5, 2018 18:37
@bagnell
Copy link
Contributor

bagnell commented Jan 5, 2018

Looks good. I just updated the spacing for the requires. Merging.

@bagnell bagnell merged commit 40c2bcc into master Jan 5, 2018
@bagnell bagnell deleted the pointPlaneProjection branch January 5, 2018 21:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants