Skip to content

ALE 2.0.0 Release Prep - #760

Merged
Kelvinrr merged 2 commits into
DOI-USGS:mainfrom
chkim-usgs:release
Nov 6, 2026
Merged

Kelvinrr merged 2 commits into
DOI-USGS:mainfrom
chkim-usgs:release

Conversation

@chkim-usgs

Copy link
Copy Markdown
Contributor

Changes for 2.0.0:

Added

  • The NEAR MSI driver now emits the single-parameter radial optical distortion (from the INS<ikid>_K1 coefficient) that ISIS models, and subtracts 0.5 from the detector center to match the ISIS 0.5-based detector-coordinate convention. The resulting CSM camera agrees with the ISIS camera to sub-pixel (cam_test pixel difference drops from ~0.7 px to ~1e-8 px on a real NEAR MSI image). #740
  • The Clementine NIR camera now emits the single-parameter radial optical distortion (k1 = -0.0006364, as in the ISIS NirCamera + RadialDistortionMap) that ISIS models, and subtracts 0.5 from its detector center to match the ISIS 0.5-based detector-coordinate convention. The resulting CSM camera agrees with the ISIS camera to sub-pixel (cam_test pixel difference drops from ~1.2 px median, ~6 px max to ~1e-7 px on a real Clementine NIR image). The UVVIS, HiRes and LWIR Clementine cameras are unchanged; only NIR is modeled here. #740
  • Quaternion reduction based on the naif spice fortran routine ck3sdn. This uses a tolerance to reduce the number of quaternions for ISDs that don't extract the exact CK times from the kernels. #747

Fixed

  • The installed CMake package config now finds its nlohmann_json dependency, so find_package(ale) works without a prior find_package(nlohmann_json). #748
  • Fixed MEX HRSC SRC detector center to match latest IAK. #753
  • Fixed Messenger MDIS to include Taylor distortion to match ISIS' implementation. #757
  • evaluateCubicHermiteFirstDeriv now checks that y has as many points as the derivatives and times, instead of reading past the end of y. #749
  • Cassini ISS drivers now produce a CSM camera that matches ISIS (ASP cam_test on a NAC image shows the ISIS-SPICE, NAIF-SPICE and PDS3 drivers agree with ISIS to ~5e-4 px). Fixes:
    • ISIS-SPICE driver: emit the radial optical distortion (was none).
    • ISIS-SPICE and NAIF-SPICE drivers: subtract 0.5 from the detector center (ISIS 0.5-based to CSM 0-based).
    • NAIF-SPICE driver: focal_length no longer raises AttributeError when INS_FOCAL_LENGTH is defined.
    • NAIF-SPICE driver: ephemeris_start_time uses the scalar from utcToEt (was a tuple). #739
  • isd_generate now fails with an error instead of writing an ISD with null required fields. #754
  • Fixed kernel quality check to allow quality keys from different missions #758

Changed

  • isd_generate: default --reduction to linear (sampling every 10th line for line-scan images with >= 1000 lines). This avoids creating oversized ISDs (tens of thousands of pose samples) for long line-scan sensors such as LRO NAC, MRO CTX, and HRSC. Pass --reduction none to disable reduction and restore per-line sampling. #744

Removed

  • Removed isis_formatter and usgscsm_formatter. Ale formatter will always be used. #737

Licensing

This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words:

This work is free and unencumbered software released into the public domain. In jurisdictions that recognize copyright laws, the author or authors of this software dedicate any and all copyright interest in the software to the public domain.

  • I dedicate any and all copyright interest in this software to the public domain. I make this dedication for the benefit of the public at large and to the detriment of my heirs and successors. I intend this dedication to be an overt act of relinquishment in perpetuity of all present and future rights to this software under copyright law.

@Kelvinrr
Kelvinrr merged commit 4c68910 into DOI-USGS:main Nov 6, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants