adflow icon indicating copy to clipboard operation
adflow copied to clipboard

Time spectral stability derivatives are broken

Open sseraj opened this issue 1 year ago • 2 comments

Description

As mentioned in the code, the time spectral stability derivatives are broken. I am creating this issue for greater visibility.

sseraj avatar Jul 06 '23 19:07 sseraj

Thanks for remingding the stability derivatives bugs. I notice the time spectral test file (test_time_spectral_naca64A010.py) used 'useexternaldynamicmesh' to define grid motion. I also tested only using 'qMode' of 'alphaMode' to run time spectral analysis but it seems has no effect for the results, which shows the same values for different timeintervals. Is using 'useexternaldynamicmesh' to define grid motion is the only way for now to run TS analysis? For another question, I can fit the data cm versus φ to get cmzq, but using cmzq as a constraint, is it also needed to evaluate it sensitive?

Besh wishes

hntter avatar Jul 07 '23 06:07 hntter

Is using 'useexternaldynamicmesh' to define grid motion is the only way for now to run TS analysis?

Yes, this is the only way that currently works.

For another question, I can fit the data cm versus φ to get cmzq, but using cmzq as a constraint, is it also needed to evaluate it sensitive?

Yes, to use cmzq as a constraint, you will also need the gradient. The stability code is differentiated with Tapenade, but it is also broken because the analysis is broken. The time spectral dynamic mesh code is not differentiated (tracked in #198).

sseraj avatar Jul 07 '23 17:07 sseraj