Skip to content

Conversation

@fso42
Copy link
Contributor

@fso42 fso42 commented Nov 18, 2025

Parallelizes the test loop in runStandardTestsCom1DFA.py to reduce wall-clock time for running standard tests, using the same CPU management approach as com1DFA.py.

  • Extracted test execution into runSingleTest() worker function
  • Implemented parallel execution using concurrent.futures.ProcessPoolExecutor
  • Dynamic worker count via cfgUtils.getNumberOfProcesses() (respects nCPU and CPUPercent config)
  • Thread-safe logging using QueueHandler/QueueListener pattern
  • Copies avaDirs to tmp folder
  • Graceful error handling - failed tests don't block successful ones
  • Added test summary showing total/successful/failed counts with error details

@fso42 fso42 self-assigned this Nov 18, 2025
@fso42 fso42 added the enhancement New feature or request label Nov 18, 2025
@qltysh
Copy link
Contributor

qltysh bot commented Nov 18, 2025

Diff Coverage: Not applicable. There was no coverage data reported for the files in this diff.

Total Coverage: This PR will not change total coverage.

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@fso42 fso42 requested a review from awirb November 19, 2025 13:45
@fso42 fso42 linked an issue Nov 19, 2025 that may be closed by this pull request
@fso42 fso42 changed the title feat(runStandardTestsCom1DFA): enable parallel execution [StandardTestsCom1]: enable parallel execution Nov 19, 2025
fso42 and others added 2 commits November 19, 2025 21:16
…logging

- Refactored script to support parallel standard test execution using `ProcessPoolExecutor`
- Introduced queue-based logging for thread-safe log handling during parallel runs.
- Organized CPU time comparison logic and added test summary reporting.

NOT WORKING FULLY YET< AS THERE ARE FAILED TESTS

feat(runStandardTestsCom1DFA): add temporary test isolation through unique directories

- Introduced `tmpTestsDir` parameter to create isolated directories for each test to avoid conflicts.
@fso42 fso42 merged commit de57a5d into master Nov 19, 2025
4 checks passed
@fso42 fso42 deleted the parallelStdTest branch November 19, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update benchmark avaInclinedPlaneEntresTest

2 participants