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

Fix velocity calculation #36

Closed
vsnever opened this issue Sep 7, 2020 · 1 comment · Fixed by #35
Closed

Fix velocity calculation #36

vsnever opened this issue Sep 7, 2020 · 1 comment · Fixed by #35

Comments

@vsnever
Copy link
Member

vsnever commented Sep 7, 2020

Currently, the parallel velocity (UA in SOLPS notation) is misinterpreted as the projection of velocity onto the poloidal direction. This is not so, parallel velocity is the projection of velocity onto the direction of the magnetic field: V|| = (VB)/|B|.

To correctly calculate the velocities we must use particle fluxes at poloidal and radial directions defined on cell faces. This will give us 4 velocity vectors for each cell at the cell faces. By averaging them we'll get the projection of velocity on the poloidal plane for each cell, Vplane. Then using Vplane, V|| and B, we can obtain Vtor (many thank to Andrey Pshenov for clarifying all this).

Note that if the flux from cell A to cell B is positive, than the velocity at the face between A and B is obtained by dividing this flux by the density at cell A, but if it's negative, than the flux must be divided by the density at cell B.

I'll add this to the code.

@vsnever
Copy link
Member Author

vsnever commented Apr 6, 2021

Fixed in #35.

@vsnever vsnever closed this as completed Apr 6, 2021
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 a pull request may close this issue.

1 participant