Skip to content
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

Negative density in the last frame when I run aurora_sim #56

Open
inwoo-song opened this issue Apr 27, 2022 · 3 comments
Open

Negative density in the last frame when I run aurora_sim #56

inwoo-song opened this issue Apr 27, 2022 · 3 comments

Comments

@inwoo-song
Copy link

Hi Francesco,
Every time I run aurora_sim, I got results of negative density in the last frame.
Even the tutorial shows the same result with the same input. It appears only in the last frame, so I think it's just minor problem but I want to make sure. Could you check this out?

image002

@fsciortino
Copy link
Owner

Hi @inwoo-song, indeed this issue is known -- but not really understood. I've never dived into this problem because, as you said, it's something that only affects the last time slice -- which can just be ignored. However, it would indeed be nice to clean this up... please let me know if you see a fix.

More broadly, note that negative densities can happen in the main part of the simulation n some extreme conditions, e.g. with very sharp gradients of diffusion in the pedestal. There is no easy solution for these cases. The algorithm used by Aurora (the Linder finite-volume modification of the Behringer-Lackner method used in STRAHL) is much better at avoiding this issue with respect to the older algorithm, but it still doesn't fundamentally prevent negative densities from occurring. I'm writing this just to give you some context, but this may never be a concern for you unless you're trying such extreme combinations of transport coefficients.

@odstrcilt
Copy link
Collaborator

The reason is that the influx in the last time step is zero. We should repair it.

@odstrcilt
Copy link
Collaborator

Here is a fix #70

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

No branches or pull requests

3 participants