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

pandas~2.0 #887

Conversation

SarahJohnsonONS
Copy link
Contributor

Version lock pandas~2.0 until #886 complete

@github-actions
Copy link

github-actions bot commented Sep 12, 2023

ubuntu-latest-python3.9- test results

  10 files    10 suites   7m 29s ⏱️
589 tests 589 ✔️ 0 💤 0
590 runs  590 ✔️ 0 💤 0

Results for commit d041eb6.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 12, 2023

ubuntu-latest-python3.11-pandas@^1.3 test results

  10 files  +  10    10 suites  +10   7m 53s ⏱️ + 7m 53s
589 tests +589  589 ✔️ +589  0 💤 ±0  0 ±0 
590 runs  +590  590 ✔️ +590  0 💤 ±0  0 ±0 

Results for commit d041eb6. ± Comparison against base commit ae5945d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 12, 2023

ubuntu-latest-python3.11-pandas~2.0 test results

  10 files    10 suites   7m 27s ⏱️
589 tests 589 ✔️ 0 💤 0
590 runs  590 ✔️ 0 💤 0

Results for commit d041eb6.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Sep 12, 2023

windows-latest-python3.11-pandas~2.0 test results

  11 files    12 suites   9m 50s ⏱️
603 tests 603 ✔️ 0 💤 0
617 runs  617 ✔️ 0 💤 0

Results for commit d041eb6.

♻️ This comment has been updated with latest results.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@NickPapONS NickPapONS left a comment

Choose a reason for hiding this comment

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

Approved. As for the errors caused by pandas 2.1.0, I came across many of the same ones locally when working on dcat distribution and had to run poetry update, so I spent some time coming up with fixes. I ended up resolving the errors on my end, but that was only on that branch which already had changes to many other files. Here is the commit where I was working on the pyright issues which mostly match the ones you listed in #886: f0f707b

Let me know what you think, I don't know if my fixes are correct since I was working locally in a branch that already had many other changes in it, so I may have approached it with the perspective of "just get it working" instead of really looking at what changed in 2.1.0 and how to fix them according to that. On one file I ended up using a # type: ignore because I couldn't find any other way to fix the error with isna not being recognised. But maybe try them on your end and see if it helps with pyright at all.

@SarahJohnsonONS SarahJohnsonONS merged commit 800072e into main Sep 13, 2023
@SarahJohnsonONS SarahJohnsonONS deleted the Version-lock-pandas-2.0-in-main-push.yaml-and-release.yaml branch September 13, 2023 13:23
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.

2 participants