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

Add plot_vdfdiff to plot_vdf.py #227

Merged
merged 10 commits into from
Aug 20, 2024
Merged

Add plot_vdfdiff to plot_vdf.py #227

merged 10 commits into from
Aug 20, 2024

Conversation

ursg
Copy link
Contributor

@ursg ursg commented Mar 20, 2024

This is mostly a copy-and-paste duplication of most of the plot_vdf code, except that it opens two files (or vlsvreader objects), can take two separate lists of cellids and then pairwise iterates through them, extracting the velocity spaces, histogrammizing them and then taking the diff of the extracted histograms.

This avoids all trouble with identifying missing blocks. Also, I suppose it has potential science applications to diff VDFs from different cells or times in their individual plasma frames.

The colour bar is linear instead of logarithmic, and centered around zero by default.

Here's an example plot from a pitch angle diffusion simulation, comparing timestep 0 and timestep 2:
plot_vdf_proton_cellid_1_t0_xy

ursg added 3 commits March 20, 2024 11:22
It can plot differences of VDFs from the same cell of different files,
or different cells of the same file.
@markusbattarbee
Copy link
Contributor

A useful tool but bloats plot_vdf horrendously. I would suggest either, preferably, adding the diff funcitonality to the existing script, or alternatively splitting this to a different file altogether. (And then pending a proper re-organization of analysator scripts to a more modular approach)

@alhom
Copy link
Contributor

alhom commented Aug 20, 2024

Split to new plot_vdfdiff file. Useful tool that should be in, but also should be revamped later.

@alhom alhom merged commit 257c78a into fmihpc:master Aug 20, 2024
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 this pull request may close these issues.

3 participants