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

KelbowVisualizer: The letters in the Legend Label are smushed together (missing spacing) #1058

Closed
lwgray opened this issue Apr 11, 2020 · 0 comments · Fixed by #1061
Closed

Comments

@lwgray
Copy link
Contributor

lwgray commented Apr 11, 2020

Describe the issue
A clear and concise description of what the bug is.
In the Kelbow visualizer I found that the Spacing between letters in the legend are absent. I also found this to be the case in the example visualization for Kelbow in both the latest and develop version of the docs. See image below
To Reproduce

from sklearn.datasets import make_blobs
from sklearn.preprocessing import StandardScaler
from sklearn.cluster import KMeans

from yellowbrick.cluster import KElbowVisualizer

# Generate sample data
centers = [[1, 1], [-1, -1], [1, -1]]
X, labels_true = make_blobs(n_samples=1500, centers=centers, cluster_std=0.3,
                            random_state=0)

Z = StandardScaler().fit_transform(X)
viz = KElbowVisualizer(KMeans())

viz.fit(Z)
viz.show()

Dataset
Did you use a specific dataset to produce the bug? Where can we access it?
yes... generated in the above code

Expected behavior
A clear and concise description of what you expected to happen.
I expect to see "elbow at k = 3, score = 264.727"

Desktop (please complete the following information):

  • OS: [e.g. macOS] MacOS
  • Python Version [e.g. 2.7, 3.6, miniconda] 3.7
  • Yellowbrick Version [e.g. 0.7] 1.1

Additional context
Add any other context about the problem here.
image

@DistrictDataLabs/team-oz-maintainers

rebeccabilbro added a commit to rebeccabilbro/yellowbrick that referenced this issue Apr 21, 2020
… and turns frameon for the legend to make it a bit easier to see
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 a pull request may close this issue.

1 participant