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

[pull] main from pydata:main #507

Merged
merged 2 commits into from
Sep 17, 2023
Merged

[pull] main from pydata:main #507

merged 2 commits into from
Sep 17, 2023

Conversation

pull[bot]
Copy link

@pull pull bot commented Sep 17, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

headtr1ck and others added 2 commits September 17, 2023 07:02
* fix several typing issues with mpl3.8

* fix scatter plot typing and remove funky pyplot import

* fix some more typing errors

* fix some import errors

* undo some typing errors

* fix xylim typing

* add forgotten import

* ignore plotting overloads because None is Hashable

* add whats-new entry

* fix return type of hist

* fix another xylim type

* fix some more xylim types

* change to T_DataArray

* change accessor xylim to tuple

* add missing return types

* fix a typing error only on new mpl

* add unused-ignore to error codes for old mpl

* add more unused-ignore to error codes for old mpl

* replace type: ignore[attr-defined] with assert hasattr

* apply code review suggestions

---------

Co-authored-by: Illviljan <14371165+Illviljan@users.noreply.github.com>
* Exclude dimensions used in faceting from squeeze

* Add unit test for facetting singleton dim

* Add bug fix to changelog

* Move test to proper class

* Update doc/whats-new.rst

* Apply suggestions from code review

* Update doc/whats-new.rst

---------

Co-authored-by: Mathias Hauser <mathause@users.noreply.github.com>
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@pull pull bot added ⤵️ pull and removed topic-plotting labels Sep 17, 2023
@pull pull bot merged commit c439432 into Illviljan:main Sep 17, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants