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

Feat matching #45

Merged
merged 29 commits into from
Mar 13, 2024
Merged

Conversation

davidsantiagoquevedo
Copy link
Member

Added features from static matching

* fix: definition of age interval when (max_val-min_val)%%setp !=0. Added option to set interval '<min_val' when min_val<0

* refac: removed test forcing interval '>max_val', when (max_val-min_val)%%step!=0. Added test for min_val>0

* refac: step=10 on PH-accept test

* feat: pulled description from previous PR

* feat: warning for module of interval by step different to zero. Updated documentation of the function

* simplified tests to control warning

* roxygen: description of warning and lower value

* fix: assert_number replaced by assert_integerish

* fix: upper = max_val for assertion on min_val

* style: documentation

* fix: warning message for (max-min)%%step!=0

* roxygen: automatic update

* fix: assert_integerish replaced by assert_int, better for integer numbers. Removed upper check for min_val to avoid redundancy with lower check for max_mal

* fix: example in documentation

* feat: test for non-integer values
…ember of couple presents outcome before censoring date
@davidsantiagoquevedo davidsantiagoquevedo merged commit 19d8bf6 into refac-effectiveness-matching Mar 13, 2024
7 checks passed
@davidsantiagoquevedo davidsantiagoquevedo deleted the feat-matching branch March 13, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant