Change PyGRB plotting scripts to read HDF files #4034
Merged
Conversation
This file contains 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
added 8 commits
July 2, 2022 16:01
pannarale
requested changes
Jul 25, 2022
pannarale
requested changes
Aug 1, 2022
pannarale
approved these changes
Sep 26, 2022
acorreia61201
pushed a commit
to acorreia61201/pycbc
that referenced
this pull request
Apr 4, 2024
* initial commit * function to extract ifos from HDF files * use ligolw for xml files * load xml table without .gz file end * function to load data from hdf files * overwrite previous load_data functions * load data for single detector chisq since plots with network chisq will be obsolete * add standard and bank chisq dof * Remove network chisq compatibility * Updates to extract_ifos * read time differently for network and single ifo, read data differently for injs * Set y_min for single ifos * Common load_data function for plotting * Update chisq veto script to work with load_data function * Update coherent ifo snr script to work with load_data function * Update null stats script to work with load_data function * Update skygrid script to work with load_data function * Update SNR timeseries script to work with load_data function * Function to plot found/missed injections in PyGRB * add decisive optimal snr to tested y variables * fix ra and dec for effective distance calculation, add decisive distance to yvars * add chirp distance to yvars * display times in relation to GRB time * remove compilation files * replace xml found/missed injection plotter with hdf injection plotter * add options to found/missed injection plots * fix codeclimate trailing whitespaces * fix wrong indentation when calculating sigmas * add missing whitespaces for codeclimate checks * fix calling of get_table function * fix visual indents * remove unused functions to load data * load data for plotting scripts individually * Deal with vetoes when loading trigger data * Add description for loading trigger function * Sort IFOs * remove unnecessary commented line * Revert back to using glue for handling MultiInspiral tables * Remove unused arguments from load_data function * Use ligo.lw.table for Table.get_table() function * Remove unwanted gz argument from utils.load_filename() * Simplify when loading ifo att * Update header * Formatting command for logging module * Add more options for possible x and y variables * Add comment for previous implementation of time-slides * add argument for dealing with injections * Remove data dictionary and load only necessary data * Fix missing colon after else statement * Remove dictionary when loading data * Fix codeclimate issues * Store only necessary values for null stat type * Add functionality for injections when loading data * Load only necessary data * Fix error when loading coincident SNR * Remove repeated line * Restructured way to load data * Add some missing units * fix code issues * Update docstring and help message to include all SNRs available * Add function to calculate coinc snr * Remove calls to numpy.asarray() for h5py file objects * Shorten coinc snr function * Fix some mistakes in code * Remove if statement when calculating coinc snr * leave file name to end with .gz when loading xml_doc * fix codeclimate issues * Keep track of injections when loading data * Remove formatting as numpy array * Load found trigger file * check options after parsing arguments * set output directory * load indices for all types of missed and recovered injections * differentiate between louder, quieter and vetoed found injs * use previous plotting settings and code structure * import numpy as np * Allow user to omit underscores for x/y variables * Add LIGO India to sites * use a function to grab subset of injections * Remove call to numpy.asarray() * remove local dictionaries * remove unused imports * remove unused imports * Update help messages for functions that handle injections only * do not store unused eta variable * Change longitude and latitude to degrees * rename function to grab injections subset * expand on comments describing indices * simplify expression for extracting inclusive/exclusive IFAR * add and remove whitespaces where necessary * add sky error to accepted values * Add/fix some comments * load spin components separately * rename load_spin data to load_spin_modulus_data Co-authored-by: Viviana Caceres <viviana.caceres@ldas-pcdev6.ligo.caltech.edu> Co-authored-by: Viviana Caceres <viviana.caceres@ldas-pcdev2.ligo.caltech.edu> Co-authored-by: Viviana Caceres <viviana.caceres@ldas-pcdev5.ligo.caltech.edu>
# for free
to join this conversation on GitHub.
Already have an account?
# 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.
No description provided.