Fix issue #201 : add PDAL reader
Implemented initial PDAL support that currently handles only LAS file formats, laying the groundwork for future enhancements and additional file format integration.
Fix: https://github.com/f3d-app/f3d/issues/201
That looks nice! If you want to only add .las suport, the next step would be to add testing:
You need to:
- Find a small license-compatible .las file
- Add a test in application/testing/CMakeLists.txt
- Add CI logic to build pdal
Of course, you may want to add more file formats, but that can come in another PR.
Nice! You guys might be interested in https://github.com/PDAL/PDAL/pull/4653 upcoming for PDAL 2.9.0 as well.
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 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.
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 ?
Hi @Husamm , do you need any help moving forward ?