Skip to content

dict unpacking broken for .plot in 3.2 #17502

Closed

Description

@story645

Edited this issue about 14 times, here is the minimal works/doesn't:

Works:

d = {4:1, 6:2}
plt.bar(d.keys(), d.values())

doesn't work

d = {4:1, 6:2}
plt.plot(d.keys(), d.values())

doesn't work as of '3.2.1.post2558.dev0+geeeafe9c2' & support for this was added in #6787

Activity

  1. changed the title [-]dict support broken in 3.2[/-] [+]dict support for categoricals broken in 3.2[/+] on May 24, 2020
  2. changed the title [-]dict support for categoricals broken in 3.2[/-] [+]dict support for units broken in 3.2[/+] on May 24, 2020
  3. changed the title [-]dict support for units broken in 3.2[/-] [+]dict unpacking for units broken in 3.2[/+] on May 24, 2020
  4. changed the title [-]dict unpacking for units broken in 3.2[/-] [+]preprocess_data broken for units in 3.2[/+] on May 24, 2020
  5. changed the title [-]preprocess_data broken for units in 3.2[/-] [+]dict unpacking broken for units in 3.2[/+] on May 24, 2020
  6. jklymak commented on May 25, 2020

    @jklymak
    Member

    Bisects to 2445465, #10928. I guess this needed user-facing tests?

  7. story645 commented on May 25, 2020

    @story645
    MemberAuthor

    @jklymak thanks for finding that, was unsure where to start on the debugging. And yeah, though not sure where to put them - there's preprocess data tests but they mock out the plot calls. But I guess can throw an integration test in there...

    Also how do you bisect?

  8. jklymak commented on May 25, 2020

    @jklymak
    Member

    Git bisect. Very helpful tool if you know something used to work!

  9. story645 commented on May 25, 2020

    @story645
    MemberAuthor

    thanks! and figured I can shove the tests into units since that's where the issue happens 😅
    never mind, not a units bug 🤦

  10. changed the title [-]dict unpacking broken for units in 3.2[/-] [+]dict unpacking broken for `.plot` in 3.2[/+] on May 25, 2020
  11. added 2 commits that reference this issue on May 25, 2020
    473d06e
    2e20e67
  12. linked a pull request that will close this issueFix dicts unpacking for `.plot` #17506on May 25, 2020
  13. added this to the v3.3.0 milestone on May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions