Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] master from OSGeo:master #98

Merged
merged 11 commits into from
Feb 10, 2025
Merged

[pull] master from OSGeo:master #98

merged 11 commits into from
Feb 10, 2025

Conversation

pull[bot]
Copy link

@pull pull bot commented Feb 10, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

- Rename 'endian' codec to 'bytes'
- Always generate a "codecs" array with the "bytes" codec in it.
  On reading, be tolerant with its absence but raise a warning
  as it being deprecated
- On reading, raise a warning on implicit groups, as being
  deprecated
- Support the 'zstd' codec, which isn't yet standardized, but is
  the default one used by Python zarr v3.0.2

Not handled: 'crc32c' and 'sharding' codecs
…nsformerFunc return FALSE as soon as one point fails to transform

Fixes #11817

Kind of a breaking change, but the current behavior was highly
inconsistent and hard to reason about.

New paragraph in MIGRATION_GUIDE.TXT:

- The following methods
  OGRCoordinateTransformation::Transform(size_t nCount, double *x, double *y,
  double *z, double *t, int *pabSuccess) and
  OGRCoordinateTransformation::TransformWithErrorCodes(size_t nCount, double *x,
  double *y, double *z, double *t, int *panErrorCodes) are modified to return
  FALSE as soon as at least one point fails to transform (to be consistent with
  the other form of Transform() that doesn't take a "t" argument), whereas
  previously they would return FALSE only if no transformation was found. When
  FALSE is returned the pabSuccess[] or panErrorCodes[] arrays indicate which
  point succeeded or failed to transform.

  The GDALTransformerFunc callback and its implementations (GenImgProjTransformer,
  RPCTransformer, etc.) are also modified to return FALSE as soon as at least
  one point fails to transform.
LIBKML: fix error when creating a Id field of type integer
Zarr V3: update with latest Zarr V3 specification
Make all Transform methods of OGRCoordinateTransformation and GDALTransformerFunc return FALSE as soon as one point fails to transform
Add 'gdal vector sql', as standalone or part of 'gdal vector pipeline'
Python bindings: add syntaxic sugar to get/set algorithm arguments + doc
@pull pull bot added the ⤵️ pull label Feb 10, 2025
@pull pull bot merged commit ebf0679 into kp-forks:master Feb 10, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13248617239

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 457 of 514 (88.91%) changed or added relevant lines in 19 files are covered.
  • 202 unchanged lines in 46 files lost coverage.
  • Overall coverage increased (+2.2%) to 72.237%

Changes Missing Coverage Covered Lines Changed/Added Lines %
alg/gdal_crs.cpp 2 3 66.67%
alg/gdalgeoloc.cpp 7 8 87.5%
gcore/gdalalgorithm.cpp 24 25 96.0%
ogr/ogrsf_frmts/libkml/ogrlibkmllayer.cpp 8 10 80.0%
ogr/ogrct.cpp 81 84 96.43%
apps/gdalalg_vector_sql.cpp 127 132 96.21%
port/cpl_compressor.cpp 5 11 45.45%
alg/gdal_rpc.cpp 9 20 45.0%
frmts/zarr/zarr_v3_codec.cpp 71 98 72.45%
Files with Coverage Reduction New Missed Lines %
apps/gdalalg_raster_mosaic.cpp 1 96.86%
third_party/LercLib/Lerc2.cpp 1 87.45%
port/cpl_vsi_error.cpp 1 54.92%
frmts/ctg/ctgdataset.cpp 1 84.65%
port/cpl_vsisimple.cpp 1 70.57%
gcore/gdalrasterband.cpp 1 77.79%
gcore/overview.cpp 1 86.82%
apps/test_ogrsf.cpp 1 72.51%
port/cpl_vsil_curl_streaming.cpp 1 86.28%
frmts/gtiff/libgeotiff/geo_tiffp.c 1 97.83%
Totals Coverage Status
Change from base Build 13228815364: 2.2%
Covered Lines: 343369
Relevant Lines: 475338

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants