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

Speed up CI #2019

Merged
merged 15 commits into from
Oct 26, 2022
Merged

Speed up CI #2019

merged 15 commits into from
Oct 26, 2022

Conversation

boeschf
Copy link
Contributor

@boeschf boeschf commented Oct 25, 2022

  • use Cmake's Ninja backend for building Arbor
  • use ccache for storing build artifacts
    • unique cache key: needs to reflect os, compiler, shared/static variant and CPU vector extensions (avx, avx2, etc)

@brenthuisman
Copy link
Contributor

@boeschf
Copy link
Contributor Author

boeschf commented Oct 25, 2022

well, the ninja stuff runs
the error came because I tried to additionally use ccache - and I am pretty sure it is due to the cache key not including the CPU architecture / SIMD vector length

@brenthuisman
Copy link
Contributor

brenthuisman commented Oct 25, 2022

The key might need to be specific for each config in the matrix. Reading the docs, I think job_id might be that unique string. [Update] On second thought, no I don't think so. Can't find a better way than ${{matrix.name}}-${{matrix.os}}-....

@boeschf boeschf changed the title Use Cmake's Ninja backend for building Arbor Speed up CI Oct 25, 2022
@thorstenhater
Copy link
Contributor

Nice. I am going to merge this and after the CI is through fold this into #1958 to check whether we get the weird behaviour.

Copy link
Contributor

@thorstenhater thorstenhater left a comment

Choose a reason for hiding this comment

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

Nice job. We should -- for the future -- investigate whether we can factor out some part of the .yml.

@thorstenhater thorstenhater merged commit fb12942 into arbor-sim:master Oct 26, 2022
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