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

Fix issue #201 : add PDAL reader #2014

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Husamm
Copy link

@Husamm Husamm commented Feb 24, 2025

Implemented initial PDAL support that currently handles only LAS file formats, laying the groundwork for future enhancements and additional file format integration.

Fix: #201

@mwestphal
Copy link
Contributor

That looks nice! If you want to only add .las suport, the next step would be to add testing:

You need to:

  1. Find a small license-compatible .las file
  2. Add a test in application/testing/CMakeLists.txt
  3. Add CI logic to build pdal

Of course, you may want to add more file formats, but that can come in another PR.

@hobu
Copy link

hobu commented Mar 3, 2025

Nice! You guys might be interested in PDAL/PDAL#4653 upcoming for PDAL 2.9.0 as well.

@Meakk
Copy link
Member

Meakk commented Mar 3, 2025

Nice! You guys might be interested in PDAL/PDAL#4653 upcoming for PDAL 2.9.0 as well.

Thanks for pointing it out.

@mwestphal
Copy link
Contributor

Hi @Husamm

Do you need help moving forward ? :)

@Husamm
Copy link
Author

Husamm commented Mar 19, 2025

Hi @mwestphal
No Thank you , I'm currently working on adding CI logic, its a bit new for me, so i have to do some learning first.
I'll update you soon.

Regards,
Husam.

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.

Add a pdal plugin relying on vtkPDALReader
4 participants