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

Updated supermic to use mpiifort with the new Intel suite they added #1414

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

wwlwpd
Copy link
Collaborator

@wwlwpd wwlwpd commented Nov 7, 2024

Issue 1413:

Resolves #1413.
Resolves #1414 (PR).

@wwlwpd wwlwpd requested a review from jasonfleming as a code owner November 7, 2024 19:17
Copy link
Collaborator

@jasonfleming jasonfleming left a comment

Choose a reason for hiding this comment

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

Hey @wwlwpd why did FFLAGS1 lose -xAVX for some of these ADCIRC versions? Is that a change for the new Intel suite?

@wwlwpd
Copy link
Collaborator Author

wwlwpd commented Nov 7, 2024

It requires mpiifort - that's effectively the only thing changed. I really just copied QBD's settings.

Copy link
Collaborator

@jasonfleming jasonfleming left a comment

Choose a reason for hiding this comment

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

Awesome thank you! :-)

@wwlwpd wwlwpd merged commit 06bae9e into master Nov 7, 2024
1 check passed
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.

update supermic (LSU) to use mpiifort
2 participants