Fix issue #201 : add PDAL reader #2014
Conversation
|
That looks nice! If you want to only add .las suport, the next step would be to add testing: You need to:
Of course, you may want to add more file formats, but that can come in another PR. |
|
Nice! You guys might be interested in PDAL/PDAL#4653 upcoming for PDAL 2.9.0 as well. |
Thanks for pointing it out. |
|
Hi @Husamm Do you need help moving forward ? :) |
|
Hi @mwestphal Regards, |
Husamm
force-pushed
the
pdal-vtk-reader-201
branch
from
8190c47 to
a90c481
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| { | ||
| "match": ".*(las|laz|pts|ptx|pcd)", | ||
| "options": { | ||
| "load-plugins": "pdal", |
There was a problem hiding this comment.
| "load-plugins": "pdal", | |
| "load-plugins": "pdal" |
| key: zlib-v1.3.1-${{runner.os}}-${{inputs.cpu}}-4 | ||
|
|
||
| # Dependents: blosc openvdb vtk | ||
| # Dependents: blosc openvdb pdal vtk |
There was a problem hiding this comment.
Yes, PDAL depends on zlib. You can see this in PDAL’s CMakeLists.txt, where it uses find_package(ZLIB REQUIRED) to locate and link against it.
.github/workflows/versions.json
Outdated
| "OpenVDB min version is not enforced by F3D code and missing from VTK code", | ||
| "Python is part of VFX reference platform (CY2025: 3.11)" | ||
| "Python is part of VFX reference platform (CY2025: 3.11)", | ||
| "PDAL is now used in VTK and should be included as a dependency." |
|
Hey @Husamm Do you need help moving forward ? |
|
Hey @mwestphal Sorry for the long delay, actually no ,i just need some time. Thanks 🙏 |
|
Hi @Husamm , do you need any help moving forward ? |
1 similar comment
|
Hi @Husamm , do you need any help moving forward ? |
4 participants
Implemented initial PDAL support that currently handles only LAS file formats, laying the groundwork for future enhancements and additional file format integration.
Fix: #201