This repository was archived by the owner on Jul 23, 2025. It is now read-only.
added PDAL loading/saving with dimension retention#6
Open
nvandamme wants to merge 2 commits into
Open
Conversation
Owner
|
Hello @nvandamme, |
Owner
|
Hello @nvandamme, I would like to know if you have ran any tests with the different formats you added? |
Author
|
Hi, only with ply and las/z, as i was focusing on these two.
Maybe there are some specific options for otherS, especially e57, though I've followed the base PDAL docs for either know dimensions and outputs options for retaining these...
My main focus was to specificaly able to retain any abstracts dimensions (scalar fields) already included in any input point cloud.
…---
Nicolas Vandamme | CEO/CTO
+352 621 41 86 07
+33 642 736 677
***@***.***
PGP Public Key ID : 7CABEC0D1973B4B8154DDA462DD266DECCC1C375<https://keys.openpgp.org/vks/v1/by-fingerprint/7CABEC0D1973B4B8154DDA462DD266DECCC1C375>
FIRIS SA | LU27948222 | RCS Luxembourg B 200097
9 rue du Laboratoire| L-1911 Luxembourg | LUXEMBOURG
________________________________
From: Anass Yarroudh ***@***.***>
Sent: Monday, July 10, 2023 8:28:55 PM
To: Yarroudh/segment-lidar ***@***.***>
Cc: Nicolas Vandamme ***@***.***>; Mention ***@***.***>
Subject: Re: [Yarroudh/segment-lidar] added PDAL loading/saving with dimension retention (PR #6)
Hello @nvandamme<https://github.com/nvandamme>, I would like to know if you have ran any tests with the different formats you added?
—
Reply to this email directly, view it on GitHub<#6 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AABRIYMWEFF5ZJ6RZKJRHILXPRCWPANCNFSM6AAAAAAZ3W3DRM>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Owner
|
The package was updated to a new version. I did not have the time to review your PR, especially that you did not run any tests with other formats. If you want always to contribute, you can adapt your changes to the new version of |
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
#4