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

Optional direct labeling in covariate balance plots #54

Merged
merged 1 commit into from
Feb 16, 2023

Conversation

ehudkr
Copy link
Collaborator

@ehudkr ehudkr commented Feb 15, 2023

The scatter plot version of the covariate imbalance plot is intended for high-dimensional covariates, forcing direct labeling without an option to opt-out can create a lot of clutter.
This PR introduces the possibility to make direct labeling of violating covariates optional to avoid possible clutter.

For the sake of completeness, the optional labeling is also implemented for the slope-graph version of the covariate balance plot.

Base automatically changed from matching-weightestimator to master February 16, 2023 14:33
Since there could be many covariates, especially in the scatter plot
that is intended for high dimension covariates, automatically labeling
the violating variables can create a lot of clutter without an opt-out
option.

Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
@ehudkr ehudkr force-pushed the imbalance-scatter-optional-labels branch from 2945c17 to 64f7f0a Compare February 16, 2023 14:54
@ehudkr ehudkr merged commit a2ec272 into master Feb 16, 2023
@ehudkr ehudkr deleted the imbalance-scatter-optional-labels branch February 16, 2023 15:00
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