Skip to content

Remove deferred loading of FieldSets #1908

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

Merged
VeckoTheGecko merged 17 commits into from
Mar 5, 2025

review feedback

20a0caa
Select commit
Loading
Failed to load commit list.
Merged

Remove deferred loading of FieldSets #1908

review feedback
20a0caa
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 5, 2025 in 0s

87.36% of diff hit (target 83.77%)

View this Pull Request on Codecov

87.36% of diff hit (target 83.77%)

Annotations

Check warning on line 546 in parcels/field.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

parcels/field.py#L543-L546 

Added lines #L543 - L546 were not covered by tests

Check warning on line 549 in parcels/field.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

parcels/field.py#L548-L549 

Added lines #L548 - L549 were not covered by tests

Check warning on line 558 in parcels/field.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

parcels/field.py#L556-L558 

Added lines #L556 - L558 were not covered by tests

Check warning on line 299 in parcels/fieldset.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

parcels/fieldset.py#L298-L299 

Added lines #L298 - L299 were not covered by tests

Check warning on line 44 in tests/test_fieldset.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

tests/test_fieldset.py#L44 

Added line #L44 was not covered by tests
View more details on Codecov