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

Xo tools #235

Open
wants to merge 72 commits into
base: master
Choose a base branch
from
Open

Xo tools #235

wants to merge 72 commits into from

Conversation

alhom
Copy link
Contributor

@alhom alhom commented May 14, 2024

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

lkotipal and others added 30 commits February 5, 2021 00:26
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)
…er), to enable auxiliary outputs from the calculation
…ing a validation case, sadly that setup is current-free and fiddly.
pyVlsv/vlsvfile.py Outdated Show resolved Hide resolved
@alhom
Copy link
Contributor Author

alhom commented Oct 4, 2024

@markusbattarbee want to have a look at the restructuring?

Copy link
Contributor

@markusbattarbee markusbattarbee left a 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 Show resolved Hide resolved

f.map_vg_onto_fg()
print('vg mapped to fg, elapsed:', time.time()-t)
argsorti=f.read_variable("CellID").argsort()
Copy link
Contributor

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 ;)

pyCalculations/null_lines.py Outdated Show resolved Hide resolved
pyCalculations/null_lines.py Outdated Show resolved Hide resolved
pyCalculations/null_lines.py Outdated Show resolved Hide resolved
pyVlsv/vlsvreader.py Outdated Show resolved Hide resolved
pyVlsv/vlsvreader.py Outdated Show resolved Hide resolved
pyVlsv/vlsvwriter.py Show resolved Hide resolved
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
Copy link
Contributor

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?

Copy link
Contributor Author

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.

file_id = int(sys.argv[1])



Copy link
Contributor

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.

Copy link
Contributor Author

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.

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