Skip to content

Missing inplace=True parameter in RINEX nav columns renaming#181

Open
FedericoPivotto wants to merge 4 commits intoStanford-NavLab:mainfrom
FedericoPivotto:patch-rinex-nav-rename
Open

Missing inplace=True parameter in RINEX nav columns renaming#181
FedericoPivotto wants to merge 4 commits intoStanford-NavLab:mainfrom
FedericoPivotto:patch-rinex-nav-rename

Conversation

@FedericoPivotto
Copy link

RINEX nav columns X, Y, Z and their deviation are not renamed.

The addition of inplace=True parameter fix the issue.

@FedericoPivotto
Copy link
Author

I have also adjust RINEX nav name convetion to make it compatible with the other library tools.

@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.66%. Comparing base (73a8198) to head (225556a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #181   +/-   ##
=======================================
  Coverage   99.66%   99.66%           
=======================================
  Files          27       27           
  Lines        3882     3882           
=======================================
  Hits         3869     3869           
  Misses         13       13           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@betaBison
Copy link
Member

Thanks for using the library @FedericoPivotto and these look like great additions. Sorry I've been traveling and it's taken a bit to check the PR. I should be able to approve and build a new version next week.

If you get to it before me, please also add your name to the contributors list.

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