-
Couldn't load subscription status.
- Fork 25
[WIP] Add curvilinear coordinates #144
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
base: main
Are you sure you want to change the base?
Conversation
|
Great, thanks! |
|
To render correctly in VisIt, just convert "spherical" to "Cartesian": |
|
Ok, it builds and is up-to-date with current AthenaPK now. |
|
How tested is this? I tried running the disk_sph.in problem, and some really weird things happen to the pressure field: disk_sph_pressure.mov |
|
Ok, I've fixed it. There was an algebraic discrepancy between the volume factors used in Athena++ and here. It works now: disk_sph_pressure.mov |
|
It looks like the ordering of the coordinates changed for cylindrical at some point, so Parthenon assumes it's R, z, phi, but this PR assumes its R, phi, z. This needs to be fixed. |
|
I don't have the capacity to fix the cylindrical coordinates at the moment, and I only need spherical for my problem, so I've extracted spherical into its own PR: #148 |
LANL has now open sourced the curvilinear coordinates code that @forrestglines developed. I was mandated to open source it on the LANL github, here: https://github.com/lanl/athenapk
This is an MR into athenapk from the branch point Forrest had. I am sure it's horribly out of date. But Philipp, if you'd like to try and bring this into alignment with the current version of AthenaPK, you now can. My apologies for my delay on this. Note that the following should be added to the license for the diff'd files, which I have not done: