Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Gitlab pipelines #934

Merged
merged 1 commit into from Aug 30, 2023
Merged

Update Gitlab pipelines #934

merged 1 commit into from Aug 30, 2023

Conversation

davschneller
Copy link
Contributor

Makes the Gitlab pipelines work.

Internally, we moved the CPU pipeline to the SCCS CI node now.

Also, the single stages are now using different node configurations.

But still, right now there are dependencies of the CPU version to be build with the respective CPU features (due to the doctest CMake script requiring them—I might file an issue for that; and the GPU version needs a CUDA runtime available which may become dificult, for now)

Internally: moved CPU pipeline to SCCS nodes

Second test

Switch to CPU compute vs CPU build

Update CI

Add custom GPU building pipeline
@krenzland
Copy link
Contributor

But still, right now there are dependencies of the CPU version to be build with the respective CPU features (due to the doctest CMake script requiring them—I might file an issue for that; and the GPU version needs a CUDA runtime available which may become dificult, for now)

What features exactly?

@davschneller
Copy link
Contributor Author

davschneller commented Aug 30, 2023

But still, right now there are dependencies of the CPU version to be build with the respective CPU features (due to the doctest CMake script requiring them—I might file an issue for that; and the GPU version needs a CUDA runtime available which may become dificult, for now)

What features exactly?

AVX or AVX2—when compiling on a Westmere CPU with optimizations targeting Haswell, they cause a Sigill quickly. Doctest executes the serial test file once—which may be compiled for the target architectures.

@davschneller
Copy link
Contributor Author

Cf. here:

execute_process(
COMMAND ${TEST_EXECUTOR} "${TEST_EXECUTABLE}" ${spec} --list-test-cases
OUTPUT_VARIABLE output
RESULT_VARIABLE result
)

@davschneller davschneller added this pull request to the merge queue Aug 30, 2023
Merged via the queue into master with commit 675b89e Aug 30, 2023
5 checks passed
@krenzland
Copy link
Contributor

Ah, but I think that is the intended behavior... :(

@krenzland krenzland deleted the davschneller/ci-test-1 branch August 31, 2023 07:28
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.

None yet

2 participants