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

2D Pitching Fixed Wing #983

Merged
merged 52 commits into from
Jul 9, 2024
Merged

2D Pitching Fixed Wing #983

merged 52 commits into from
Jul 9, 2024

Conversation

ET14
Copy link
Contributor

@ET14 ET14 commented Feb 26, 2024

Add capabilities to read a pitch angle time series from user provided text file for the fixed wing actuator case.
The implementation is intended to work also for "2D" cases and still misses the implementation of a 2D Gaussian.

@mbkuhn
Copy link
Contributor

mbkuhn commented Feb 26, 2024

@ET14 great job getting this set up! I've added a few commits for the sake of passing the CI tests and correcting a print statement. If there are more changes you plan to make, please perform "git pull" before making the changes. I can also add the 2D Gaussian -- we need that for other uses as well.

Copy link
Contributor

@tonyinme tonyinme left a comment

Choose a reason for hiding this comment

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

It's looking good. Here are some minor comments.

amr-wind/wind_energy/actuator/wing/ActuatorWing.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/wing_ops.H Outdated Show resolved Hide resolved
@tonyinme
Copy link
Contributor

@ET14 great job getting this set up! I've added a few commits for the sake of passing the CI tests and correcting a print statement. If there are more changes you plan to make, please perform "git pull" before making the changes. I can also add the 2D Gaussian -- we need that for other uses as well.

@mbkuhn what do you think is the best way to implement the 2D Gaussian? We would need some information to orient the Gaussian in the right direction.

…tatement for aoa update (pitch table is now in deg rather than in rad)
amr-wind/wind_energy/actuator/ActSrcLineOp.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/actuator_utils.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/actuator_utils.H Outdated Show resolved Hide resolved
Copy link

github-actions bot commented May 4, 2024

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

… span to ensure force conservation 2) Option to disable fz actuator force 3) Option to use a velocity magnitude of unity for force calculation
@ET14
Copy link
Contributor Author

ET14 commented May 8, 2024

@tonyinme do we want to generalize the options to

  • disable certain force components
  • calculate the actuator force with the freestream velocity magnitude
  • use the 2D Gaussian in "3D simulations" where another axis than the y-direction is the spanwise direction?
    In its current form the implementation works for our specific problem, but is definitely not ready to be merged.

@mbkuhn
Copy link
Contributor

mbkuhn commented Jun 4, 2024

@ET14 @tonyinme I will try to get this polished up for merging soon. I'll be making direct edits to the branch, but feel free to make comments on my work.

@mbkuhn
Copy link
Contributor

mbkuhn commented Jun 5, 2024

@ET14 I believe I have generalized the capabilities from your work. I put a new reg test in test/test_files/act_pitching_wing_2D that should serve as an example for the input arguments. Please try out that case and see if it does what you need it to. It would be great if you could add a pitch_timetable file to that test case, too. I will probably add a unit test or two before merging, but I think it's ready otherwise. By the way, the "get_either" and "query_either" changes are for something Tony requested -- they weren't needed to get this to work.

@ET14
Copy link
Contributor Author

ET14 commented Jun 5, 2024

@mbkuhn Thank you for restructuring this, looks great! I will test all the flags again and also add a pitch input table.

…g span for the test case to match the domain width in the periodic direction
@ET14
Copy link
Contributor Author

ET14 commented Jun 20, 2024

@mbkuhn I re-ran some of my previous pitching wing test cases with the code version which you restructured two weeks ago and I obtain the same results. So the 2D Gaussian seems to work fine and also disabling the vertical forcing produced the results I expected.
I added a pitch input time series to the test case and adjusted the wing span in this example to match the domain width in the periodic direction.
From my side this pull request has all the capabilities it needs. Feel free to merge it when you think it is ready. Thank you again for all your help!

amr-wind/wind_energy/actuator/wing/ActuatorWing.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/fixed_wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/fixed_wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/fixed_wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/wing_ops.H Outdated Show resolved Hide resolved
test/test_files/act_pitching_wing_2D/sineWave.txt Outdated Show resolved Hide resolved
@mbkuhn
Copy link
Contributor

mbkuhn commented Jun 28, 2024

reg tests passed. last thing is unit tests

* was not used in actual code, only in unit_tests
* flat_plate and fixed_wing inputs are not similar enough to warrant another function
@mbkuhn mbkuhn marked this pull request as ready for review July 5, 2024 14:52
Copy link
Contributor

@marchdf marchdf left a comment

Choose a reason for hiding this comment

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

Nice work! I left some minor suggestions.

amr-wind/wind_energy/actuator/wing/wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/fixed_wing_ops.H Outdated Show resolved Hide resolved
amr-wind/wind_energy/actuator/wing/wing_ops.H Outdated Show resolved Hide resolved
@mbkuhn
Copy link
Contributor

mbkuhn commented Jul 9, 2024

requested changes addressed

@mbkuhn mbkuhn merged commit b8ab898 into Exawind:main Jul 9, 2024
13 checks 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