PDAL / python

PDAL's Python Support
Other
116 stars 34 forks source link

Pythonic Pipeline creation #91

Closed gsakkis closed 2 years ago

gsakkis commented 2 years ago

This PR extends pdal.Pipeline to provide easier pipeline specification and composition. The README is updated with a new section and the Reading using Numpy Arrays example has been updated to use the new API.

The changes are backwards compatible with only one minor exception: the json parameter of pdal.Pipeline() has been renamed to spec (since it can now handle more than a JSON string).

This PR is built on top of #89; check out this diff for the changes in this branch.

abellgithub commented 2 years ago

Can you please merge master into this so that I can see what's here?

spolloni commented 2 years ago

@gsakkis can you comment on how this relates to #42 ?

gsakkis commented 2 years ago

@spolloni this is loosely inspired by #42 in terms of goals and high-level API but the details and actual implementation are completely separate. Once this is merged I'd expect pio to be deprecated/removed sometime in the future.