Skip to content

Conversation

@mvanzulli
Copy link
Member

@mvanzulli mvanzulli commented Jun 29, 2022

A larger tolerance is defined for simpleWindTurbine.mcase since is not converging for some specific cpus. This PR seek to close #514

@mvanzulli
Copy link
Member Author

mvanzulli commented Jun 29, 2022

Hey there @JoaquinViera you can execute this in your own pc. Allegedly we have found with @jorgepz some minor numerical results in the report due to different harreares, or CPU architectures. However the numerical results were converging to a norm of $\Delta_u \approx 1e-12$ and $ \Delta_r \approx 1e-6 $, so we have decided to just increase the residual tolerance in force since the aim of this test is to check residual force tolerance (tolU=0).

Thanks for reporting it 👍

@mvanzulli mvanzulli marked this pull request as ready for review June 29, 2022 20:46
@jorgepz jorgepz changed the title Defining a larger tolerance in simple wind turbine example Fix simple wind turbine example tolerances Jun 29, 2022
@mvanzulli mvanzulli merged commit 3a28f26 into master Jul 1, 2022
@jorgepz jorgepz deleted the mvanzulli/fixWindTurbine branch July 1, 2022 12:19
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.

non convergence in simple wind Turbine example

4 participants