Skip to content

feat: update lasrc-rs post-alignment of C-to-Rust code - #63

Merged
ceholden merged 19 commits into
mainfrom
chore/rust-align-from-pr
Nov 4, 2026
Merged

ceholden merged 19 commits into
mainfrom
chore/rust-align-from-pr

Conversation

@ceholden

@ceholden ceholden commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

What I am changing

This PR updates the Rust port of LaSRC to include a handful of final "C to Rust alignment" PRs that ensured the Rust port faithfully matched the C code version:

These PRs have been merged into the long lived espa-surface-reflectance:refactor branch, and then migrated to https://github.com/NASA-IMPACT/lasrc-rs.

The https://github.com/NASA-IMPACT/lasrc-rs will publish a Conda package to our HLS channel, but for now I want to test the code using this repository. Once it's stable and we have our version strategy locked in, we'll issue a lasrc-rs==v1.0.0 and switch to the Conda package release for that version.

@ceholden ceholden changed the title Chore/rust align from pr fix: LaSRC Rust to C alignment Sep 16, 2026
@ceholden
ceholden force-pushed the chore/rust-align-from-pr branch from afb39a8 to 4630305 Compare September 18, 2026 16:02
@ceholden
ceholden force-pushed the chore/rust-align-from-pr branch from 75ae895 to 0e7e667 Compare October 7, 2026 16:03
@ceholden ceholden changed the title fix: LaSRC Rust to C alignment feat: LaSRC Rust to C alignment Oct 28, 2026
@ceholden
ceholden force-pushed the chore/rust-align-from-pr branch from 5cb17fa to 4aa8791 Compare November 4, 2026 21:58
@ceholden ceholden changed the title feat: LaSRC Rust to C alignment feat: update lasrc-rs post-alignment of C-to-Rust code Nov 4, 2026
@ceholden
ceholden merged commit cc583b6 into main Nov 4, 2026
7 checks passed
@ceholden
ceholden deleted the chore/rust-align-from-pr branch November 4, 2026 22:09
@ceholden
ceholden restored the chore/rust-align-from-pr branch November 5, 2026 15:48
@ceholden
ceholden deleted the chore/rust-align-from-pr branch November 5, 2026 15:48
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.

1 participant