Skip to content

Nesi#73

Closed
dkhutch wants to merge 2 commits intoACCESS-NRI:masterfrom
dkhutch:nesi
Closed

Nesi#73
dkhutch wants to merge 2 commits intoACCESS-NRI:masterfrom
dkhutch:nesi

Conversation

@dkhutch
Copy link

@dkhutch dkhutch commented Feb 20, 2026

Changes to AVX vectorization flags for compiling on NeSI.

@dougiesquire
Copy link
Collaborator

Thanks for this PR @dkhutch! Unfortunately our CI doesn't run from forks (security) so I've cherry-picked your change into a branch in this repo and opened a new PR with the same change here: #74

Next time we can give you write access so you can submit your PR from a branch on this repo - just open an issue if you have any other proposed changes.

I'll close this PR now and move over to #74

@dkhutch
Copy link
Author

dkhutch commented Feb 24, 2026

Thanks Dougie. Ok, next time I'll see if I can do it without making a fork. I might need to ask some follow-up basic questions though, because I'm not very familiar with github pull requests.

@dougiesquire
Copy link
Collaborator

All good. And to be clear, the approach you took of making a fork is the most common way. We're just a bit cautious because our CI submits things to Gadi.

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.

2 participants