Skip to content

Conversation

jbangelo
Copy link
Collaborator

@jbangelo jbangelo commented Sep 3, 2025

The final chunk of changes from #122 🎉

Simply put this removes the remaining functionality that depends on the C library and breaks the dependency on the swiftnav-sys crate. I wouldn't say that we'll never want this functionality in Rust, but these modules have not been meaningfully used up till now and have non-trivial C code backing them so would require an undue amount of work to re-write.

What's been changed

  • Crate level documentation has been updated to not reference removed functionality, and to give a description of functionality originally missed
  • Minimum supported Rust version (MSRV) has been bumped up to reflect reality (also fix CI build to check MSRV still builds)

What's been removed

  • ephemeris module - Decoding and calculating Satellite positions
  • geoid module - Calculating difference between Ellipsoid and Geoid
  • ionosphere module - Decoding and evaluating Klobuchar ionosphere model
  • troposphere module - Calculating troposphere delays
  • navmeas module - Datatypes related to storing GNSS measurments
  • solver module - Single epoch position solver
  • All conversions to/from swiftnav_sys types, all of these were private

What's been added

  • N/A

@jbangelo jbangelo marked this pull request as ready for review September 3, 2025 02:04
@jbangelo jbangelo requested a review from a team as a code owner September 3, 2025 02:04
Copy link
Contributor

@JADC362 JADC362 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Joseph for working on this!

@jbangelo jbangelo merged commit d914ea4 into master Sep 4, 2025
6 checks passed
@jbangelo jbangelo deleted the jbangelo/riir-cleanup branch September 4, 2025 01:49
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.

3 participants