Skip to content

ci: Exclude StrawLineFitterTest from debug build tests#5489

Merged
andiwand merged 3 commits into
acts-project:mainfrom
andiwand:ci-debug-build-exclude-strawlinefittertest
May 22, 2026
Merged

ci: Exclude StrawLineFitterTest from debug build tests#5489
andiwand merged 3 commits into
acts-project:mainfrom
andiwand:ci-debug-build-exclude-strawlinefittertest

Conversation

@andiwand

@andiwand andiwand commented May 21, 2026

Copy link
Copy Markdown
Contributor

Currently this thing alone runs 25 minutes in our CI.

--- END COMMIT MESSAGE ---

debug build

347/347 Test #135: StrawLineFitterTest ..................   Passed  1409.57 sec

normal build

358/358 Test #135: StrawLineFitterTest ..................   Passed   24.03 sec

Ultimately we should reconsider if this is a test or how this is implemented

cc @junggjo9 @benjaminhuth

@andiwand andiwand added this to the next milestone May 21, 2026
@benjaminhuth

Copy link
Copy Markdown
Member

Maybe we can think where to put this executable, because its not really a unit test I think. Maybe to the benchmarks?

@github-actions github-actions Bot added Component - Core Affects the Core module Infrastructure Changes to build tools, continous integration, ... Seeding labels May 21, 2026
@andiwand andiwand marked this pull request as ready for review May 21, 2026 13:12
@github-actions

github-actions Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

📊: Physics performance monitoring for a267443

Full contents

physmon summary

@sonarqubecloud

Copy link
Copy Markdown

@andiwand andiwand merged commit 0d43864 into acts-project:main May 22, 2026
46 of 47 checks passed
@andiwand andiwand deleted the ci-debug-build-exclude-strawlinefittertest branch May 22, 2026 05:47
@andiwand andiwand modified the milestones: next, v46.7.0 May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component - Core Affects the Core module Infrastructure Changes to build tools, continous integration, ... Seeding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants