Skip to content

Conversation

@jcrist
Copy link
Member

@jcrist jcrist commented Jul 7, 2025

This moves the code and infra for running upstream test suites with cuml.accel into the same directory as all the other cuml accel tests (python/cuml/cuml_accel_tests, not python/cuml/cuml/accel/tests). This has bothered my for a while, but is now motivated by generalizing the infra to run other test suites besides sklearn's (e.g. #6680)

Everything specific to sklearn is in the python/cuml/cuml_accel_tests/upstream/scikit-learn directory, everything general is one directory up in the python/cuml/cuml_accel_tests/upstream directory.

This moves the code and infra for running upstream test suites with
`cuml.accel` into the same directory as all the other cuml accel tests
(`python/cuml/cuml_accel_tests`, not `python/cuml/cuml/accel/tests`).
This has bothered my for a while, but is now motivated by generalizing
the infra to run other test suites besides sklearn's.
@jcrist jcrist requested review from a team as code owners July 7, 2025 20:31
@jcrist jcrist requested review from AyodeAwe, bdice and cjnolet July 7, 2025 20:31
@github-actions github-actions bot added Cython / Python Cython or Python issue ci labels Jul 7, 2025
@jcrist jcrist requested review from csadorf and removed request for cjnolet July 7, 2025 20:32
@jcrist jcrist added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 7, 2025
Copy link
Contributor

@csadorf csadorf left a comment

Choose a reason for hiding this comment

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

Fine from my side.

@jcrist
Copy link
Member Author

jcrist commented Jul 7, 2025

/merge

@rapids-bot rapids-bot bot merged commit 1513f84 into rapidsai:branch-25.08 Jul 7, 2025
72 of 73 checks passed
@jcrist jcrist deleted the move-cuml-accel-upstream-tests branch July 7, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants