Skip to content

Convert conftest-patch.py to a pytest plugin for pandas testing - #22334

Closed
mroeschke wants to merge 4 commits into
NVIDIA:mainfrom
mroeschke:p3/cudf/pandas_test_suite_pt3
Closed

Convert conftest-patch.py to a pytest plugin for pandas testing#22334
mroeschke wants to merge 4 commits into
NVIDIA:mainfrom
mroeschke:p3/cudf/pandas_test_suite_pt3

Conversation

@mroeschke

Copy link
Copy Markdown
Contributor

Description

Towards #20859

This PR converts our conftest-patch.py to a conftest_cudf_pandas.py pytest plugin file that can be used as pytest -p conftest_cudf_pandas to patch pandas testing utilities and dynamically add our xfail or skip markers instead of modifying the existing pandas' conftest.py

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@mroeschke mroeschke self-assigned this Apr 29, 2026
@mroeschke mroeschke added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Apr 29, 2026
@copy-pr-bot

copy-pr-bot Bot commented Apr 29, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added Python Affects Python cuDF API. cudf.pandas Issues specific to cudf.pandas labels Apr 29, 2026
@mroeschke

Copy link
Copy Markdown
Contributor Author

/ok to test bd33450

@GPUtester GPUtester moved this to In Progress in cuDF Python Apr 29, 2026
@mroeschke

Copy link
Copy Markdown
Contributor Author

/ok to test 6bba91b

@mroeschke

Copy link
Copy Markdown
Contributor Author

/ok to test 504c64f

@mroeschke

Copy link
Copy Markdown
Contributor Author

I'll wait until we get the pandas test suite to green before moving forward to more easily see if these changes impact the run

galipremsagar added a commit that referenced this pull request May 1, 2026
…22339)

## Description
closes #20859

While working on #22334, it's
easier to validate that the changes are net even with the current state
of the pandas test suite runs if this job is green

## Checklist
- [ ] I am familiar with the [Contributing
Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md).
- [ ] New or existing tests cover these changes.
- [ ] The documentation is up to date with these changes.

---------

Co-authored-by: GALI PREM SAGAR <sagarprem75@gmail.com>
@mroeschke
mroeschke changed the base branch from pandas3 to main May 19, 2026 21:44
@mroeschke

Copy link
Copy Markdown
Contributor Author

/ok to test bd59d70

@mroeschke

Copy link
Copy Markdown
Contributor Author

Superseded by #22643

@mroeschke mroeschke closed this May 22, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in cuDF Python May 22, 2026
@mroeschke
mroeschke deleted the p3/cudf/pandas_test_suite_pt3 branch July 30, 2026 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cudf.pandas Issues specific to cudf.pandas improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Affects Python cuDF API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants