Skip to content

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

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.

Sign up for GitHub

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

Open
FedericoPivotto wants to merge 4 commits into
base: main
Choose a base branch
from
Open

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a74df84
Add missing inplace=True in data column renaming
FedericoPivotto Jan 16, 2026
225556a
Adjust RINEX nav columns name convetion
FedericoPivotto Jan 16, 2026
427fd87
Update pylint.yml disabling Poetry parallel install
FedericoPivotto Feb 12, 2026
eed082b
Update build.yml disabling Poetry parallel install
FedericoPivotto Feb 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
with:
virtualenvs-create: true
virtualenvs-in-project: true
installer-parallel: true # default option selected currently
installer-parallel: false # default option selected currently
# Load cached environment, if it exists
- name: Load cached poetry environment
id: cached-poetry-dependencies
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pylint.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
with:
virtualenvs-create: true
virtualenvs-in-project: true
installer-parallel: true # default option selected currently
installer-parallel: false # default option selected currently
- name: Load cached poetry environment
id: cached-poetry-dependencies
uses: actions/cache@v4
Expand Down
6 changes: 3 additions & 3 deletions gnss_lib_py/parsers/rinex_nav.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -216,9 +216,9 @@ def _get_ephemeris_dataframe(self, rinex_path, constellations=None):
# Rename Keplerian orbital parameters to match a GLP standard
data.rename(columns={'M0': 'M_0', 'Eccentricity': 'e', 'Toe': 't_oe', 'DeltaN': 'deltaN', 'Cuc': 'C_uc', 'Cus': 'C_us',
'Cic': 'C_ic', 'Crc': 'C_rc', 'Cis': 'C_is', 'Crs': 'C_rs', 'Io': 'i_0', 'Omega0': 'Omega_0'}, inplace=True)
data.rename(columns={'X': 'sv_x_m', 'dX': 'sv_dx_mps', 'dX2': 'sv_dx2_mps2',
'Y': 'sv_y_m', 'dY': 'sv_dy_mps', 'dY2': 'sv_dy2_mps2',
'Z': 'sv_z_m', 'dZ': 'sv_dz_mps', 'dZ2': 'sv_dz2_mps2'}, )
data.rename(columns={'X': 'x_sv_m', 'dX': 'dx_sv_mps', 'dX2': 'dx2_sv_mps2',
'Y': 'y_sv_m', 'dY': 'dy_sv_mps', 'dY2': 'dy2_sv_mps2',
'Z': 'z_sv_m', 'dZ': 'dz_sv_mps', 'dZ2': 'dz2_sv_mps2'}, inplace=True)
return data, data_header

def get_iono_params(self, rinex_header, constellations=None):
Expand Down
Loading