Assessing balance with only select columns #39
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Sometimes the
is_continuous
is not precise enough to identify continuous and categorical variables, leading to unwieldy plots. Also sometimes it's useful to only plot the balance on a few specific covariates of interest after matching.Changes
In order to enable that, I have added a new argument
columns: List[str]
to theMatcher.compare_continuous
andMatcher.compare_categorical
methods. Whencolumns
is passed in, we do not verify if the column is continuous or categorical but we do remove columns in theself.exclude
set.There is no actual change within the plotting for loops (as indicated below). The changes are simply due to the removal of indentation.
Verification
I ran the example notebook with and without columns and they work as intended. See nbviewer.