Skip to content

New compasUtils file#1467

Open
reinhold-willcox wants to merge 1 commit intodevfrom
compas_post_processing_utils
Open

New compasUtils file#1467
reinhold-willcox wants to merge 1 commit intodevfrom
compas_post_processing_utils

Conversation

@reinhold-willcox
Copy link
Collaborator

Added my own compasUtils file, which I use for lots of post-processing and testing. It was previously in misc/unsupported_utils, so I am hoping to get it elevated to the level of the other utils.

This contains two major functions that help a lot with inspecting COMPAS data.

  • printCompasDetails: takes the hdf5 output (optionally with specific seeds or masks) and prints a pandas df of the whole output (including units). Makes it visually much easier to analyze the output from a jupyter notebook.
  • getEventStrings: takes the hdf5 output and generates a string per seed that summarizes the events in the binary. Events include SMT, CEE, SNe, and mergers, and these are listed chronologically with a funny but simple syntax (described in the function). Combined with np.unique(), this is a great way to quickly see which binary channels dominate and which are rare.

I expect that there will be changes requested, so it is probably not yet ready to be merged in, but happy to hear feedback on how to improve / align these scripts with the other python utilities.

@github-actions
Copy link

✅ COMPAS Build Successful!

Item Value
Commit 98df732
Logs View workflow

Detailed Evolution Plot

Click to view evolution plot


Generated by COMPAS CI

Copy link
Collaborator

@ilyamandel ilyamandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I leave the review to folks who use python (@SimonStevenson ? @avigna ? ).

Consider including supported python tools in the documentation in https://compas.readthedocs.io/en/latest/pages/User%20guide/Post-processing/post-processing.html

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.

2 participants