Skip to content

Conversation

@csadorf
Copy link
Contributor

@csadorf csadorf commented Apr 11, 2025

Create a draft PR to run CI on test infrastructure improvements. Do not merge.

@csadorf csadorf added the DO NOT MERGE Hold off on merging; see PR for details label Apr 11, 2025
@csadorf csadorf self-assigned this Apr 11, 2025
@copy-pr-bot
Copy link

copy-pr-bot bot commented Apr 11, 2025

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 conda conda issue Cython / Python Cython or Python issue labels Apr 11, 2025
@csadorf
Copy link
Contributor Author

csadorf commented Apr 11, 2025

/ok to test 5a2fd63

@csadorf csadorf force-pushed the work/improve-test-robustness branch from 5a2fd63 to 56aeac7 Compare April 11, 2025 20:25
@csadorf
Copy link
Contributor Author

csadorf commented Apr 11, 2025

/ok to test 56aeac7

@csadorf csadorf force-pushed the work/improve-test-robustness branch from 56aeac7 to b1c0fb8 Compare April 16, 2025 15:36
csadorf added 8 commits April 22, 2025 16:03
Added sections on test organization, accuracy testing, memory usage considerations, and best practices for writing tests. Included detailed recommendations for using fixtures, parametrization, and hypothesis for test input generation. This update aims to improve the clarity and effectiveness of testing strategies within the codebase.
Included a section detailing how to run tests from the python/cuml/ directory, including common pytest commands and options.
Introduced a new section outlining three levels of test parameters: unit, quality, and stress tests.
The test should be removed with 25.08, not 24.08.
Introduced a new memory profiler plugin that tracks memory usage during test execution. The plugin records memory usage at the start and end of each test, reporting significant memory consumption. Updated the test configuration to include the new plugin for enhanced memory monitoring.
Updated the memory profiler plugin to utilize GPU memory statistics and issue warnings for high memory usage during test execution. Removed the previous memory tracking methods and streamlined the reporting mechanism to focus on peak GPU memory usage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cython / Python Cython or Python issue DO NOT MERGE Hold off on merging; see PR for details

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant