Skip to content
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.

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

Make required_keys always a set #383

Merged
merged 7 commits into from
Mar 2, 2024

Docstring and typing for check_keys.

25a7180
Select commit
Loading
Failed to load commit list.
Merged

Make required_keys always a set #383

Docstring and typing for check_keys.
25a7180
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 2, 2024 in 0s

88.13% of diff hit (target 74.92%)

View this Pull Request on Codecov

88.13% of diff hit (target 74.92%)

Annotations

Check warning on line 212 in scopesim/effects/apertures.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/effects/apertures.py#L212

Added line #L212 was not covered by tests

Check warning on line 274 in scopesim/effects/electronic.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/effects/electronic.py#L274

Added line #L274 was not covered by tests

Check warning on line 34 in scopesim/effects/rotation.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/effects/rotation.py#L34

Added line #L34 was not covered by tests

Check warning on line 595 in scopesim/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/utils.py#L595

Added line #L595 was not covered by tests

Check warning on line 597 in scopesim/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/utils.py#L597

Added line #L597 was not covered by tests

Check warning on line 606 in scopesim/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

scopesim/utils.py#L605-L606

Added lines #L605 - L606 were not covered by tests