Skip to content

Commit ec463fd

Browse files
Remove test kwarg from ensemble branch
Remove ensemble stuff from builer too
1 parent 4502c94 commit ec463fd

File tree

3 files changed

+5
-18
lines changed

3 files changed

+5
-18
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ jobs:
1212
runs-on: ubuntu-latest
1313
steps:
1414
- uses: actions/[email protected]
15+
with:
16+
fetch-depth: 0
1517
- uses: actions/setup-python@v6
1618
- uses: pre-commit/[email protected]
1719

@@ -27,14 +29,14 @@ jobs:
2729
environments: ${{ matrix.environment }}
2830
frozen: true # This seems hacky?
2931

30-
- name: Clean git workspace
32+
- name: Clean workspace
3133
run: |
3234
git clean -fdx
3335
git reset --hard HEAD
3436
3537
- name: Run tests and generate coverage report
3638
run: |
37-
pixi clean && pixi run -e ${{ matrix.environment }} test-cov
39+
pixi run -e ${{ matrix.environment }} test-cov
3840
3941
- name: Upload code coverage
4042
uses: codecov/codecov-action@v5

src/access_nri_intake/source/builders.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1046,19 +1046,4 @@ def parser(cls, file: str) -> dict:
10461046
]
10471047
)
10481048

1049-
if cls.ensemble:
1050-
with open_dataset_cached(
1051-
file,
1052-
decode_cf=False,
1053-
decode_times=False,
1054-
decode_coords=False,
1055-
) as ds:
1056-
member_id = ds.attrs.get("realization_index", None)
1057-
if member_id is None:
1058-
raise ParserError(
1059-
f"Cannot determine member for file {file} - "
1060-
"realization_index attribute missing"
1061-
)
1062-
ncinfo_dict["member"] = f"r{int(member_id)}"
1063-
10641049
return ncinfo_dict

tests/test_builders.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
(["roms"], "ROMSBuilder", {}, 4, 4, 1),
3636
(["access-esm1-6"], "AccessEsm16Builder", {"ensemble": False}, 20, 20, 7),
3737
(["woa"], "WoaBuilder", {}, 7, 7, 2),
38-
(["cmip6"], "Cmip6Builder", {"ensemble": False}, 74, 73, 14),
38+
(["cmip6"], "Cmip6Builder", {}, 74, 73, 14),
3939
],
4040
)
4141
@pytest.mark.filterwarnings("ignore:Time coordinate does not include bounds")

0 commit comments

Comments
 (0)