search

digitalearthpacific / dep-tools

Processing tools for Digital Earth Pacific
MIT License
1 stars 0 forks source link

0.3.0 candidate #50

Closed jessjaco closed 6 months ago

jessjaco commented 6 months ago

Breaking changes, I have envisioned this as the next minor version. It should implement most of the unpulled changes from main, though I haven't checked the last couple of commits. The two primary focuses were

  1. Reduce wrappers around things that should be passed as kwargs to e.g. odc.stac.load.
  2. Reduce complexity by trying to enforce single responsibility as much as possible.

These things are most evident in the loader. I removed most of the named arguments from OdcLoader so they can be passed directly to load. I also created a new abstraction called a SearchLoader which allows separation of stac searching and loading the searched items.

I made similar changes to the writers, though I tried (and haven't yet thoroughly tested) to mimic the existing functionality of DsWriter so it's a little easier to use.

I added some minor functionality here and there, particularly automatic pathrow-based searching to the landsat searcher.

I fixed some bugs in the antimeridian fixes.

I added some tests.

I started paying more robust attention to no data values, based on #31. Sensitivity to this when writing should be checked, and more tests should be written for this piece.

This should serve as the base for development and use going forward.

alexgleith commented 6 months ago

I've merged in the main branch... but we should be a bit cautious, because it was a messy merge!

I'd love to get this merge into main, so we only have one branch to worry about @jessjaco :-)