-
Notifications
You must be signed in to change notification settings - Fork 10
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
Advection-diffusion convergence issue #161
Comments
I had a look at this. I think it is unrealistic to start with a case which has an infinite gradient. I have an example notebook that looks at advection starting from a finite time and looking at the change over time and with advection. I'll upload and you can review. |
Keen to have a look! |
@bknight1 Take a look at Jupyterbook/Notebooks/Examples-Convection/Ex_AdvectionDiffusion_1dBlock.py. I'm careful to start from a finite time after the step-function initial condition is applied so that it can be represented reasonably well by the mesh (maybe less so for the simplex example shown). Move it / diffuse it and compare before the analytic solution becomes invalid. |
Makes sense! I'll update the script and see how it performs |
With increasing resolution the error is not showing a convergence order:
The errors are concentrated at the diffusive boundary:
unknown (T) degree comparison:
diffusivity (D) value comparison:
To test any updates, the script to check convergence is found here
The text was updated successfully, but these errors were encountered: