-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST/VIZ: add test for legend colors for DataFrame with duplicate column labels #11136 #32451
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gabrielvf1 for the PR.
pandas/tests/plotting/test_frame.py
Outdated
@@ -3316,6 +3316,22 @@ def test_missing_markers_legend_using_style(self): | |||
self._check_legend_labels(ax, labels=["A", "B", "C"]) | |||
self._check_legend_marker(ax, expected_markers=[".", ".", "."]) | |||
|
|||
def test_colors_of_columns_with_same_name( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something like test_legend_colors_duplicate_labels ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the reply, i have just made the changes . Can you check if is everything right now?
I have just made the changes. Can you check if is everything right now? Really thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gabrielvf1 for those updates.
Fix were made, the for syntax is better now and now im using methods from matplotlib like get_color() and get_legend(). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gabrielvf1 lgtm pending green
Great thanks @gabrielvf1 |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff