f3d icon indicating copy to clipboard operation
f3d copied to clipboard

Fix issue #201 : add PDAL reader

Open Husamm opened this issue 10 months ago • 9 comments

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

Husamm avatar Feb 24 '25 13:02 Husamm

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.

mwestphal avatar Mar 01 '25 12:03 mwestphal

Nice! You guys might be interested in https://github.com/PDAL/PDAL/pull/4653 upcoming for PDAL 2.9.0 as well.

hobu avatar Mar 03 '25 19:03 hobu

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

Thanks for pointing it out.

Meakk avatar Mar 03 '25 20:03 Meakk

Hi @Husamm

Do you need help moving forward ? :)

mwestphal avatar Mar 15 '25 08:03 mwestphal

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.

Husamm avatar Mar 19 '25 06:03 Husamm

Hey @Husamm

Do you need help moving forward ?

mwestphal avatar Apr 21 '25 06:04 mwestphal

Hey @mwestphal

Sorry for the long delay, actually no ,i just need some time.

Thanks 🙏

Husamm avatar Apr 23 '25 05:04 Husamm

Hi @Husamm , do you need any help moving forward ?

mwestphal avatar Jun 09 '25 18:06 mwestphal

Hi @Husamm , do you need any help moving forward ?

mwestphal avatar Jun 18 '25 05:06 mwestphal