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

Update csv_stack test for pandas >= 2.1 #653

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

mferrera
Copy link
Collaborator

@mferrera mferrera commented Jan 24, 2024

Resolves #634

Pandas 2 preserves the column ordering now, presumably as a bug fix, so have adapted to the test to not expect the old column ordering. The ordering will remain different from Python 3.8 because this change occurred between Pandas 2.0.3 and later versions (2.1, 2.2). I could not find the exact Pandas issue.

Pandas 2 preserves the column ordering now, presumably as a bug fix, so
have adapted to the test to not expect the old column ordering.
@mferrera mferrera self-assigned this Jan 24, 2024
@mferrera mferrera marked this pull request as ready for review January 24, 2024 06:55
@mferrera mferrera requested a review from alifbe January 24, 2024 06:56
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ea85898) 86.11% compared to head (4132dfb) 86.11%.

❗ Current head 4132dfb differs from pull request most recent head 318e8aa. Consider uploading reports for the commit 318e8aa to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #653   +/-   ##
=======================================
  Coverage   86.11%   86.11%           
=======================================
  Files          50       50           
  Lines        7058     7058           
=======================================
  Hits         6078     6078           
  Misses        980      980           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mferrera mferrera requested a review from rnyb January 24, 2024 07:02
@mferrera mferrera changed the title Update csv_stack for pandas 2 Update csv_stack test for pandas >= 2.1 Jan 24, 2024
Copy link
Collaborator

@rnyb rnyb left a comment

Choose a reason for hiding this comment

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

LGTM

@mferrera mferrera merged commit 3e40c54 into equinor:main Jan 24, 2024
5 checks passed
@mferrera mferrera deleted the fix-pandas-2 branch January 24, 2024 08:48
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.

Test failure in pandas>=2
3 participants