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

Add MHD module into main repository #264

Merged
merged 44 commits into from
Apr 22, 2024
Merged

Conversation

rfj82982
Copy link
Collaborator

No description provided.

Copy link
Contributor

@mathrack mathrack left a comment

Choose a reason for hiding this comment

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

A few comments. I will look at the mhd file later.

examples/OTV-Ozage-Tang-vortex-2D/CMakeLists.txt Outdated Show resolved Hide resolved
examples/OTV-Ozage-Tang-vortex-2D/CMakeLists.txt Outdated Show resolved Hide resolved
examples/OTV-Ozage-Tang-vortex-2D/input_test.i3d Outdated Show resolved Hide resolved
examples/OTV-Ozage-Tang-vortex-2D/input_test.i3d Outdated Show resolved Hide resolved
src/Case-Channel.f90 Outdated Show resolved Hide resolved
src/implicit.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
src/mptool.f90 Outdated Show resolved Hide resolved
@rfj82982
Copy link
Collaborator Author

All unnecessary code has been removed.

src/Case-Channel.f90 Show resolved Hide resolved
src/Case-Channel.f90 Show resolved Hide resolved
src/mhd.f90 Show resolved Hide resolved
src/mhd.f90 Outdated Show resolved Hide resolved
src/mhd.f90 Show resolved Hide resolved
src/mhd.f90 Outdated Show resolved Hide resolved
src/mhd.f90 Outdated Show resolved Hide resolved
@mathrack
Copy link
Contributor

We should merge #267 first then this PR. I have rebased the present PR on top of the PR removing kind-specific functions

@rfj82982
Copy link
Collaborator Author

Ok I have merged #267. Sylvain still wanted some modifications on the MHD examples. Shall we merge and after do the changes once Jian gives the case with the channel?

@mathrack
Copy link
Contributor

mathrack commented Apr 22, 2024

I think the PR can be merged, and a later PR will polish the cases... But we could wait as well for a finalized version of the mhd cases.

@rfj82982
Copy link
Collaborator Author

I'll check with Jian when the channel case is ready. If takes few more days I'll merge and refactor the MHD examples in a dedicated PR.

@slaizet slaizet merged commit 4e0112b into xcompact3d:master Apr 22, 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.

4 participants