amr-wind icon indicating copy to clipboard operation
amr-wind copied to clipboard

Adding a dynamic stress (wave model) to wall_models

Open ayalamanuel opened this issue 1 year ago • 1 comments

Summary

we have introduced a dynamic stress as a wall model. the dynamic stress calculates the stress due to waves. a new struct was implemented in ShearStressSimple.H where the wave stress is calculated. a new header (MOSD.H) was created with the calculations of the stress. the WallFunction.cpp has been modified accordingly.

No unit testing or regression testing has been done.

Pull request type

Please check the type of change introduced:

  • [ ] Bugfix
  • [ x] Feature
  • [ ] Code style update (formatting, renaming)
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes
  • [ ] Documentation content changes
  • [ ] Other (please describe):

Checklist

The following is included:

  • [ ] new unit-test(s)
  • [ ] new regression test(s)
  • [ ] documentation for new capability

This PR was tested by running:

  • the unit tests
    • [ ] on GPU
    • [ ] on CPU
  • the regression tests
    • [ ] on GPU
    • [ ] on CPU

Additional background

Issue Number:

ayalamanuel avatar Sep 04 '24 19:09 ayalamanuel

@ayalamanuel do you have any updates on this?

marchdf avatar Sep 11 '24 17:09 marchdf

@marchdf Old files have been updated. Unit test, regression test and documentation have been added. Thanks!

ayalamanuel avatar Sep 18 '24 06:09 ayalamanuel

I've made the modifications to all the comments made @marchdf

ayalamanuel avatar Sep 18 '24 17:09 ayalamanuel

A meta comment here: to get past the CI check, the first thing you need to do is format your code. You can do that automatically with clang-format: https://exawind.github.io/amr-wind/developer/coding_guidelines.html#style-guide-recommendations

marchdf avatar Sep 18 '24 18:09 marchdf

Can I do anything to help push this through? We are super close I think.

marchdf avatar Oct 09 '24 16:10 marchdf

@marchdf @moprak-nrel sorry for taking so long with this! Hopefully it's almost finished

ayalamanuel avatar Oct 14 '24 13:10 ayalamanuel

can you resolve those documentation conflicts?

marchdf avatar Oct 14 '24 18:10 marchdf

seems like the documentation issues are because I used and outdated theory.rst file. How can I update that file in my branch? then I can just add the once again the new part of documentation from my implementation

ayalamanuel avatar Oct 17 '24 09:10 ayalamanuel

seems like the documentation issues are because I used and outdated theory.rst file. How can I update that file in my branch? then I can just add the once again the new part of documentation from my implementation

I would rebase your branch with the latest amr-wind main and resolve any conflicts there. You'll need to do this before the PR is ready for merging anyway.

moprak-nrel avatar Oct 17 '24 16:10 moprak-nrel

@ayalamanuel I think we are close! I updated some things/fixed conflicts.

marchdf avatar Nov 20 '24 17:11 marchdf

This runs fine on GPU.

marchdf avatar Nov 20 '24 22:11 marchdf