Skip to content

test(cudf-polars): compare local and upstream test coverage - #24164

Draft
vyasr wants to merge 28 commits into
NVIDIA:mainfrom
vyasr:codex/fresh-cudf-polars-build
Draft

vyasr wants to merge 28 commits into
NVIDIA:mainfrom
vyasr:codex/fresh-cudf-polars-build

Conversation

@vyasr

@vyasr vyasr commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR adds a reproducible comparison of cudf-polars source coverage from the local tests and the upstream Polars test suite. The command records per-test coverage in separate data files and produces line-level JSON, an HTML summary, local-test classifications, and a ranked queue of possible local-to-upstream duplicate pairs. Each pair includes shared and local-only source lines, representative test contexts, and whether the upstream node appears in the maintained GPU-execution list. The reports record the cuDF revision, installed Polars version, matching upstream Polars checkout revision, and suite exit statuses.

The queue identifies tests for behavioral review, not tests approved for deletion. The developer guide documents close behavioral matches, the backend checks they do not replace, and examples of misleading overlap. The command retains diagnostic reports when a suite fails but still exits unsuccessfully. This PR does not remove tests or change Polars dependencies, GPU injection, or upstream test selection.

Checklist

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

@copy-pr-bot

copy-pr-bot Bot commented Sep 14, 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-polars Issues specific to cudf-polars labels Sep 14, 2026
@vyasr
vyasr force-pushed the codex/fresh-cudf-polars-build branch 2 times, most recently from 85f70d0 to 5bda998 Compare September 16, 2026 22:29
@vyasr vyasr added tests Unit testing for project improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 9, 2026
@vyasr
vyasr force-pushed the codex/fresh-cudf-polars-build branch from 5bda998 to 4213f83 Compare October 12, 2026 20:02
@vyasr
vyasr force-pushed the codex/fresh-cudf-polars-build branch from 4213f83 to 04c3c2e Compare November 2, 2026 20:15
@vyasr vyasr changed the title test(cudf-polars): rationalize upstream Polars GPU gating test(cudf-polars): compare local and upstream test coverage Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cudf-polars Issues specific to cudf-polars improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Affects Python cuDF API. tests Unit testing for project

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant