Skip to content

[Doc]: Pyplot Axes – tuple or list? #26508

Closed

Description

@EngineerKhan

Documentation Link

https://matplotlib.org/stable/api/_as_gen/matplotlib.pyplot.axes.html

Problem

It reads: "4-tuple of floats rect = [left, bottom, width, height]. "

Shouldn't it be a list?

Suggested improvement

Perhaps we should go for the tuples here.

Activity

  1. rcomer commented on Aug 12, 2023

    @rcomer
    Member

    Thanks @EngineerKhan, I think either will work, but I agree this docstring should be self-consistent. Changing it as you say would make it match fig.add_axes (which it calls underneath). Would you like to make a PR?

  2. EngineerKhan commented on Aug 13, 2023

    @EngineerKhan
    ContributorAuthor

    @rcomer I have made the PR. Please check if it sounds good.

  3. rcomer commented on Aug 14, 2023

    @rcomer
    Member

    @EngineerKhan I cannot see a pull request from you.

  4. EngineerKhan commented on Aug 14, 2023

    @EngineerKhan
    ContributorAuthor

    @rcomer Maybe I done it wrongly? Can you please check it again?

  5. rcomer commented on Aug 14, 2023

    @rcomer
    Member

    @EngineerKhan it looks like you still need to create the pull request from your branch. If you go here then you should see a "New pull request" button.

  6. EngineerKhan commented on Aug 14, 2023

    @EngineerKhan
    ContributorAuthor

    @rcomer My bad. I have made one now. Hope it's the right way to do it.

  7. linked a pull request that will close this issueReplaced list with tuple in pyplot for axes #26521on Aug 14, 2023
  8. added this to the v3.8.0 milestone on Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions