-
Notifications
You must be signed in to change notification settings - Fork 32
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
Xo tools #235
base: master
Are you sure you want to change the base?
Xo tools #235
Conversation
Still needs testing, might have discrepancies with single cell calculation.
…ne-cube collision finding that did approximately nothing (0.5*sqrt(3) is still a good threshold that catches the missing pieces)
…rid support in vlsvWriter
…er), to enable auxiliary outputs from the calculation
…ing a validation case, sadly that setup is current-free and fiddly.
…ake slicing op faster
@markusbattarbee want to have a look at the restructuring? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You want comments? You got comments!
examples/sidecar.py
Outdated
|
||
f.map_vg_onto_fg() | ||
print('vg mapped to fg, elapsed:', time.time()-t) | ||
argsorti=f.read_variable("CellID").argsort() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
argsorti
and rev_argosorti
are also not used
If this is supposed to be an example file, I think it dould do a lot better job of explaining what it does ;)
scripts/inspect_neutrals.py
Outdated
from matplotlib.lines import Line2D | ||
import time | ||
|
||
# https://stackoverflow.com/questions/13685386/matplotlib-equal-unit-length-with-equal-aspect-ratio-z-axis-is-not-equal-to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This script has old crud in it, and no explanation of what it does :)
Is it honestly something that another user could pick up and use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should go to the papers folders - it has tooling to reproduce some of the plots in the XO paper. Adding descriptions.
scripts/neutral_lines.py
Outdated
file_id = int(sys.argv[1]) | ||
|
||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here and in other places: I dislike uploading files to the analysator repo where there are hardcoded input or output file names or directories. I suggest you insert boilerplate argument handling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This particular file can actually bite the dust. Or maybe rather, I could repurpose this one as an example on producing LMN-XO sidecars.
So this could go in sometime!
Plenty of stuff, used in the XO paper.
pyCalculations/null_lines.py contains wraps the FOTE method - it has some extra outputs for plotting the estimated null lines, which are then discarded in the corresponding datareducers. This function also has a vectorized golden ratio optimizer, which could be its own script... some time.
Plenty of reducers for MGA and MDD, and these guys could use the long-discussed reducer caching... but that can happen sometime in the future as well.
Contains the PR:
https://github.com/fmihpc/analysator/pull/232/files