-
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 edge_profiles data in IMAS DD #4
Comments
@eldond Are you actively working on this? I can take over if you have not started this one. |
@anchal-physics I wanted to but then got sidetracked. Go ahead and take it. I'll finish the extrapolations. |
Grid and subset visualization has been implemented. Please comment if you would like any more tweaks or features in this visualization. Key issues have been found in SOLPS2IMAS. I'll reopen the issue there. Note that the issue of clipping on left margin is me not knowing well about how to save plots in Julia, it is not an issue of the recipe afaik. |
This is very nice. Not only does the visualization look great, it revealed problems with SOLPS2IMAS that were quite difficult to identify otherwise. It's very easy to be off-by-one with indexing these grids because of the difference between cells and faces and because of the guard cells at the edges that are sometimes counted and sometimes not. |
Finished adding 2D Visualization for data.It is still very naive implementation, so this can be improved better in future, but for now, we should be good to go. For example, when overlaying grid on a heatmap, the grid does not appear everywhere which I am not sure why it is happening. This can be a enhancement issue in future. Added examples alsoI've added some minimum working examples on how to plot. Checkout using plotlyjs() as well for interactivity. Please ask for more specific examples if something looks unclear. |
If this were a python plot with some disappearing elements, I would suggest that the zorder of plot elements needs to be controlled. That is, the lines might be there, but behind the colored patches. Anyway, nice work. This is very useful for verifying our data. I was thinking of storing the extended mesh in |
I realized there was no issue for the 2D plot of 1D profiles, so I opened #9. |
Reply to previous comment, according to the manual, grid_ggd[:] are made an array to store time varying grids, so it won't be standard use of the data structure to store extended mesh in grid_ggd[2]. I present two other alternatives: 1. Multiple spacesThe extended grid can be stored in space[2] within the same grid_ggd as that is supposed to represent different spaces.
would plot the two spaces in the same plot with the existing code. 2. Additional grid_subsetAnother possibility (as we discussed earlier, keeping it in writing here) is that we simply add extended grid as a new grid_subset. This will involve adding new nodes, edges, and cells to the existing space but referred to only by the new grid_subset (say with index 200). This solution has the benefit that the extended mesh cells can be "connected" to SOL cells using boundary attributes while this can not be done if the extended mesh goes in a separate space. In this case, the whole grid including the extension can be plotted by
If only the SOLPS grid needs to be plotted, one can give the grid_subset 5
Currently the above two commands do the same thing. If only the extended grid is to be plotted, one can do
|
Yes, you're right, and it's good to write it down so I don't forget again. |
The text was updated successfully, but these errors were encountered: