Splitting plot functions away from calculations/themis_observation.py #448
Open
lassejsc wants to merge 7 commits intofmihpc:masterfrom
Open
Splitting plot functions away from calculations/themis_observation.py #448lassejsc wants to merge 7 commits intofmihpc:masterfrom
lassejsc wants to merge 7 commits intofmihpc:masterfrom
Conversation
virtual_observations
Member
|
Please check plotting functionality, add an example plot here and an example plotting call to the functions. |
lassejsc
commented
Mar 23, 2026
| cmapuse=pt.plot.get_cmap("Blues") | ||
|
|
||
| cax = ax.pcolormesh(xi,yi,vi.T, norm=matplotlib.colors.LogNorm(vmin=vmin,vmax=vmax), vmin=vmin, vmax=vmax, cmap=cmapuse, shading='flat') | ||
| cax = ax.pcolormesh(xi,yi,vi.T, norm=matplotlib.colors.LogNorm(vmin=vmin,vmax=vmax), cmap=cmapuse) |
Contributor
Author
There was a problem hiding this comment.
Note the change here, shading flat complained about the input shapes for this function and vmin,vmax complained about already being in the norm input, this made it plot with least amount of effort.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added also some functionality to plot to file with
outputfileand whether to draw withdrawandnooverwrite. Addedncontourskwarg for phaespace_contour allowing to set the level of contours to plot and additionally via this also fixed the colorbar ticks and formatting. (Not sure why the font changes with the format but maybe someone else can fiddle with it)Example plots given by
phasespace_contour

phasespace_helistyle


detector