Skip to content

Commit

Permalink
Python >= 3.9. CI also test 3.13
Browse files Browse the repository at this point in the history
Windows Python 3.13 not in GA cache yet...
  • Loading branch information
scivision committed Jan 9, 2025
1 parent 95ef1d8 commit 08698c1
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,15 @@ jobs:

strategy:
matrix:
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
os: [ubuntu-latest]
experimental: [true]
include:
- os: windows-latest
python-version: "3.12"
experimental: false
- os: macos-latest
python-version: "3.12"
python-version: "3.13"
experimental: false

steps:
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ classifiers = [
"Programming Language :: Python :: 3",
"Topic :: Scientific/Engineering :: Atmospheric Science"
]
requires-python = ">=3.8"
requires-python = ">=3.9"
dynamic = ["readme", "version"]
dependencies = ["python-dateutil", "numpy", "xarray", "hatanaka", "ncompress", "netcdf4"]

Expand Down
2 changes: 1 addition & 1 deletion src/georinex/nav3.py
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ def _sparefields(cf: list[str], sys: str, N: int) -> list[str]:
cf = cf[:30]
elif sys == "J":
# QZSS
if N==28:
if N == 28:
N = 29
cf = cf[:29]
elif N == 29:
Expand Down
12 changes: 6 additions & 6 deletions src/georinex/tests/test_nav3.py
Original file line number Diff line number Diff line change
Expand Up @@ -259,15 +259,15 @@ def test_ionospheric_correction():

assert nav.attrs["ionospheric_corr_GAL"] == approx([0.1248e03, 0.5039, 0.2377e-01])

def test_missing_fields():
"""Tests the conditions when missing fields exist within rinex data.

"""
nav = gr.load(R / "BRDC00IGS_R_20201360000_01D_MN.rnx", use='E', verbose=True)
def test_missing_fields():
"""Tests the conditions when missing fields exist within rinex data."""
nav = gr.load(R / "BRDC00IGS_R_20201360000_01D_MN.rnx", use="E", verbose=True)
# missing fields should be interpreted as zero and not NaN
# no NaN values should exist when loading the provided rinex file
assert nav.to_dataframe().isna().sum().sum() == 0


def test_missing_fields_end_of_line():
"""Test when missing fields are at the end of the line.
Expand All @@ -276,5 +276,5 @@ def test_missing_fields_end_of_line():
value should be interpretted as 0.
"""
nav = gr.load(R / "BRDM00DLR_R_20130010000_01D_MN.rnx", use='J', verbose=True)
assert nav.to_dataframe()["FitIntvl"].to_list() == [0.,0.]
nav = gr.load(R / "BRDM00DLR_R_20130010000_01D_MN.rnx", use="J", verbose=True)
assert nav.to_dataframe()["FitIntvl"].to_list() == [0.0, 0.0]

0 comments on commit 08698c1

Please sign in to comment.