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

MAINT: Replace iteritems with items #2683

Merged
merged 1 commit into from Oct 27, 2022
Merged

MAINT: Replace iteritems with items #2683

merged 1 commit into from Oct 27, 2022

Conversation

joelostblom
Copy link
Contributor

@joelostblom joelostblom commented Sep 22, 2022

iteritems is raising a warning in pandas >=1.5 as it is now deprecated. The recommended replacement is items.

@lbittarello
Copy link

lbittarello commented Oct 25, 2022

Pandas has now removed iteritems (49244), so it'd be great to merge this! @jakevdp

@joelostblom
Copy link
Contributor Author

@mattijn This seems to be a harmless change to me, do you think we can merge this PR?

Copy link
Contributor

@mattijn mattijn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, all good👍

@joelostblom joelostblom merged commit 58cbef1 into master Oct 27, 2022
@mattijn mattijn mentioned this pull request Nov 23, 2022
4 tasks
mattijn pushed a commit that referenced this pull request Jan 20, 2023
* DOC: remove unused section

* Disable uri-reference format check in jsonsschema (#2771)

* Disable uri-reference format check. Consistently use same validator across codebase

* Remove validation in SchemaInfo as not used anywhere and it referenced the wrong jsonschema draft

* Add compatibility for older jsonschema versions

* Improve comments

* Simplify validate_jsonschema

* Replace `iteritems` with `items` due to pandas deprecation (#2683)

* Add changelog entry.

* Bump version.

* Run black and flake8.

* Pin selenium in CI.

Co-authored-by: Jake VanderPlas <jakevdp@google.com>
Co-authored-by: Stefan Binder <binder_stefan@outlook.com>
Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>
mattijn pushed a commit to mattijn/altair that referenced this pull request Jan 21, 2023
* DOC: remove unused section

* Disable uri-reference format check in jsonsschema (vega#2771)

* Disable uri-reference format check. Consistently use same validator across codebase

* Remove validation in SchemaInfo as not used anywhere and it referenced the wrong jsonschema draft

* Add compatibility for older jsonschema versions

* Improve comments

* Simplify validate_jsonschema

* Replace `iteritems` with `items` due to pandas deprecation (vega#2683)

* Add changelog entry.

* Bump version.

* Run black and flake8.

* Pin selenium in CI.

Co-authored-by: Jake VanderPlas <jakevdp@google.com>
Co-authored-by: Stefan Binder <binder_stefan@outlook.com>
Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>
mattijn added a commit that referenced this pull request Jan 21, 2023
* include an altairfuturewarning

* deprecate vega 5 wrappers and render function

* deprecate vegalite 3 wrappers and render function

* use AltairDeprecationWarning

* fix typo in v5 warning

* remove mentioning alternative for vega wrappers

* Backport bug fixes for a 4.2.1 release (#2827)

* DOC: remove unused section

* Disable uri-reference format check in jsonsschema (#2771)

* Disable uri-reference format check. Consistently use same validator across codebase

* Remove validation in SchemaInfo as not used anywhere and it referenced the wrong jsonschema draft

* Add compatibility for older jsonschema versions

* Improve comments

* Simplify validate_jsonschema

* Replace `iteritems` with `items` due to pandas deprecation (#2683)

* Add changelog entry.

* Bump version.

* Run black and flake8.

* Pin selenium in CI.

Co-authored-by: Jake VanderPlas <jakevdp@google.com>
Co-authored-by: Stefan Binder <binder_stefan@outlook.com>
Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* include note in releases change log

Co-authored-by: Jan Tilly <jan.tilly@quantco.com>
Co-authored-by: Jake VanderPlas <jakevdp@google.com>
Co-authored-by: Stefan Binder <binder_stefan@outlook.com>
Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>
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 this pull request may close these issues.

None yet

3 participants