-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make 2D visualizations for core_profiles #9
Comments
I agree, we need to move SD4SOLPS.core_profile_2d() to GGDUtils (along with a helper function check_rho_1d or we can copy this small functionality inside core_profile_2d() to break dependence). With this reorganzation, we can keep all interpolating codes in GGDUtils.jl/interpolations.jl With a deeper look, core_profile_2d is really a general interpolation from (r,z) coordinates to rho_n 1d with last line special implementation for a core_profile. The real work of core_profile creation is happening anyway in extrapolate_core function in SD4SOLPS.jl/supersize_profile.jl. In the end, it would be nice to have an overloaded interp function such that:
returns a callable function
returns a callable function Finally, assuming core_profile data has been filled using SD4SOLPS.extrapolate_core, we can use
returns a callable function
returns a callable function P.S. Maybe a new issue should be created for this while reserving this space for core_profile visualization. |
I agree with your design/reorganization logic. Open new issues as needed to keep track of ideas or just implement them at your discretion. |
The 1D core profile has to be projected to 2D R,Z space.
SD4SOLPS.core_profile_2d()
can be used for this purpose, although importing it from SD4SOLPS when GGDUtils is imported by SD4SOLPS might be a problem. Maybe the function needs to be moved to GGDUtils? Or maybe this particular plot belongs in SD4SOLPS instead of in GGDUtils?The text was updated successfully, but these errors were encountered: