Allow field groups in SCREEN_OUTPUT (e.g. RMS_RES like for HISTORY_OUTPUT) #1587
Conversation
pcarruscag
changed the title
|
This pull request fixes 5 alerts when merging 525ce91 into c204683 - view on LGTM.com fixed alerts:
|
There was a problem hiding this comment.
Awesome! Love it 💐 I tend to write every RMS, so this will ease things a bit. I guess this also works for adjoint cases which is one thing less to care about in a setup 👍
https://github.com/su2code/SU2/pull/1571/files#diff-7d139cfb7a735e5c085a26f115306f13b8209745edf69dc56281269478784972 from #1571 would be a nice application as it would cover a primal and an adjoint case (momentum residuals are not present in the current setup though)
(I guess the order of RMS is changed for your adapted testcase: Now it is rho, rhoE, rhoU, rhoV instead of rho, rhoU, rhoV, rhoE ... but no values changed)
…u2code/SU2 into allow_groups_for_screen_output
|
This pull request fixes 5 alerts when merging d8b83f3 into c204683 - view on LGTM.com fixed alerts:
|
bigfooted
left a comment
bigfooted
left a comment
There was a problem hiding this comment.
great work, thanks for the follow up!
Small change for consistency with work done in #1552 to address #1493
Useful at least to get all the residuals with little work...