Skip to content

Increase ITK coverage time out for tests#2251

Open
blowekamp wants to merge 1 commit intoSimpleITK:mainfrom
blowekamp:itk_coverage_timeout
Open

Increase ITK coverage time out for tests#2251
blowekamp wants to merge 1 commit intoSimpleITK:mainfrom
blowekamp:itk_coverage_timeout

Conversation

@blowekamp
Copy link
Copy Markdown
Member

The itkComposeBigVectorImageFilterTest was timing out.

@blowekamp blowekamp requested a review from dave3d February 26, 2025 15:15
Copy link
Copy Markdown
Member

@dave3d dave3d left a comment

Choose a reason for hiding this comment

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

LGTM!

The itkComposeBigVectorImageFilterTest was timing out.
@blowekamp blowekamp force-pushed the itk_coverage_timeout branch from adcf826 to ac950c0 Compare February 26, 2025 19:35
@blowekamp blowekamp requested a review from dzenanz February 26, 2025 21:03
@blowekamp
Copy link
Copy Markdown
Member Author

The itkComposeBigVectorImageFilterTest test is still timing out. Maybe some profiling needs to be done on the test to see why it 's so slow.

Copy link
Copy Markdown
Collaborator

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

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

LGTM

@dzenanz
Copy link
Copy Markdown
Collaborator

dzenanz commented Feb 26, 2025

Maybe it is using a lot of memory, thus causing use of swap file in the CI runner? If we request RUN_SERIAL, that might help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants