Skip to content

Add dummy controller for primary simulation#12

Open
karimali5 wants to merge 77 commits intoigridSGS-KSHfrom
igridSGS-KA
Open

Add dummy controller for primary simulation#12
karimali5 wants to merge 77 commits intoigridSGS-KSHfrom
igridSGS-KA

Conversation

@karimali5
Copy link
Copy Markdown

@karimali5 karimali5 commented Sep 19, 2025

Changes in here

When wind-angle controller is activated for both the precursor and the primary simulations, the controller action for the primary starts to strongly oscillate leading to strong pressure oscillations which contaminate the solution.

For example in the below figure, the blue markers show the control action of the primary simulation when it has its own independent controller versus that of the precursor simulation (black). The control action of the primary simulation oscillates between the controller trigger (around 0.1) and ~15 degrees.

Controller_action

This behavior leads to nonphysical velocities as indicated by the hub-height slice of the normalized streamwise velocity ($u/G$).

uslice-controller

The solution here is to introduce the flag 'dummy_controller' (true or false) in the &CONTROL namelist. When activated for the primary simulation, a dummy wind-angle controller is created which points to the controller of the precursor simulation.

When the controller action is being calculated (in 'update_RHS_control' being called from 'popRHS_stuff.F90'), the dummy controller mimics the action of the precursor controller and applies this action to the primary flow field rather than calculating its own action based on the primary flow field.

The resulting control action and wind-speed field are shown above labeled 'dummy controller'.

@karimali5 karimali5 requested a review from kirbyh September 19, 2025 16:56
@karimali5 karimali5 changed the base branch from main to igridSGS-KSH September 19, 2025 17:07
kirbyh and others added 30 commits February 19, 2026 14:09
Rebase to igridSGS-KSH
…t to avoid deadlock that occasionally happens
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.

3 participants