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 SU2-python 2.0 #1359

Merged
merged 3 commits into from
Aug 24, 2021
Merged

Update SU2-python 2.0 #1359

merged 3 commits into from
Aug 24, 2021

Conversation

Nicola-Fonzi
Copy link
Contributor

@Nicola-Fonzi Nicola-Fonzi commented Aug 24, 2021

Proposed Changes

Just small changes suggested by @TobiKattmann (Thank you) in #1357

PR Checklist

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with the '-Wall -Wextra -Wno-unused-parameter -Wno-empty-body' compiler flags, or simply --warnlevel=2 when using meson).
  • My contribution is commented and consistent with SU2 style.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp) , if necessary.

@Nicola-Fonzi Nicola-Fonzi mentioned this pull request Aug 24, 2021
5 tasks
Copy link
Contributor

@TobiKattmann TobiKattmann left a comment

Choose a reason for hiding this comment

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

Thanks @Nicola-Fonzi 💐

As my review was too late, it would have been totally fine to just do a local commit and share (and reference it) with the next PR 👍

Just one more question: Can the features that should be run only in serial mode, be run in parallel by accident without throwing some error like The combination ... only allows for serial execution of the code.. If not I would appreciate those kinds of checks that prevent use out of spec.

@Nicola-Fonzi
Copy link
Contributor Author

I definitely closed #1357 too fast, so no problem. I tried to implement all your suggestions. Regarding the possibility of running by error in parallel the structural solver, this is not possible. That part of the code is meant to solve a small system of structural equations, so it is only implemented in serial mode.

@Nicola-Fonzi Nicola-Fonzi merged commit ac042cb into su2code:develop Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants