Add PointInTimeCriterion.forward_fill property #237

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

glichtner merged 1 commit into main from forward-filling

Nov 25, 2024

Merged

Add PointInTimeCriterion.forward_fill property #237

feat: add PointInTimeCriterion.forward_fill property

This check has been archived and is scheduled for deletion. Learn more about checks retention

Codecov / codecov/patch failed Nov 25, 2024 in 0s

73.33% of diff hit (target 87.61%)

View this Pull Request on Codecov

73.33% of diff hit (target 87.61%)

Annotations

Check warning on line 36 in execution_engine/omop/criterion/point_in_time.py

codecov / codecov/patch

execution_engine/omop/criterion/point_in_time.py#L36

Added line #L36 was not covered by tests

Check warning on line 109 in execution_engine/omop/criterion/point_in_time.py

codecov / codecov/patch

execution_engine/omop/criterion/point_in_time.py#L107-L109

Added lines #L107 - L109 were not covered by tests

View more details on Codecov