Skip to content

Conversation

linev
Copy link
Member

@linev linev commented Jun 27, 2025

  1. In root/io/perf directory run most macros - not only julius.C.
  2. In ReadBuffer.C/WriteBuffer.C exclude tests with direct use of TStreamerInfo methods - which are now private
  3. ShortTest/HardTest takes too long and creating huge files - therefore put under if(very_long_tests)
  4. Replicate Makefile logic in root/io/perf/slowreading - first create marker file and then run all combinations checking that time is not much differ
  5. Enable root/io/perf/stl tests with TLorenzVector - were not running with Makefile
  6. Enable root/io/perf/simple tests - were not running with Makefile
  7. Convert root/io/perf/userdatasets, all tests are failing and taking extremely long time, therefore use if(very_long_tests)

linev added 3 commits June 27, 2025 16:56
Run julius.C
Also run ReadBuffer/WriteBuffer tests - without direct sinfo methods
SergeiShortTest and SergeiHardTest not included - while consume a lot of time
Like in Makefile, first run reference and measure time
After that run all tests and check them with reference time
linev added 2 commits June 27, 2025 21:30
Just run all macros in compiled mode
Only exclude vectorsptrtclass - it uses private members
@linev linev force-pushed the roottest_io_perf branch from f8b47cb to 454f922 Compare June 27, 2025 19:32
Copy link

github-actions bot commented Jun 27, 2025

Test Results

    20 files      20 suites   3d 19h 30m 14s ⏱️
 3 107 tests  3 107 ✅ 0 💤 0 ❌
60 545 runs  60 545 ✅ 0 💤 0 ❌

Results for commit fcc3a37.

♻️ This comment has been updated with latest results.

linev added 2 commits June 30, 2025 10:06
Remove unused reference files,
Most of tests are failing - only atlas file has two tests passing
Actual failures can be investigated later
While tests taking extremely long time - put then under `very_long_tests` condition
Copy link
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

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

Thanks.

@linev linev merged commit ccc9fc7 into root-project:master Jul 3, 2025
24 of 25 checks passed
@linev linev deleted the roottest_io_perf branch July 3, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants