Updating fork #9
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
Co-authored-by: Michael <michael.patel@ligo.org>
* Add irregular bin spacing. * Remove empty line * Remove empty line
* Changes in arm length value * Changes in the equation of the orbit For equal arm length 0.03 AU
* update for new gwosc links * update * fix * fix
* don't try and copy coinc attributes * fix unicode
* We should be consistent about units * Do not use numpy vectorize * remove more numpy vect * no commit * cc
…3403) * Add try/except block around skcuda.fft import to ensure ImportError * Address CC line length issue
* Added high-frequency cutoff to pycbc_single_template. * Added check if high-frequency cutoff is given.
* Multi-injection test for pycbc live * update run.sh * Updated test to address Tito's comments * testing * Please work * Fixed snr * Updated injection generation to reproduce an error * Fixed multi-inj test to not rely on h5py 5.10 * Addressed Tito's comments Co-authored-by: max.trevor <max.trevor@ldas-pcdev6.ligo.caltech.edu>
* guess channel name from version * try alternate
…3411) * duration weighted throughput plot * test * fix * use harmonic mean
* Document options to read science and veto segments from XML files * Remove misleading sentence about Python 3 compatibility
…3414) * Started adding high-frequency cutoff. * Implemented high-frequency cutoffs. Changes: -Added a --f-upper option to pycbc_make_skymap. It passes those to pycbc_make_single template and SingleCoincForGraceDB. -Added high-frequency cutoff to SingleCoincForGraceDB. PSDs may now be truncated at a high-frequency cutoff. * Wrapped lines for CodeClimate. * Finished implementing a high-frequency cutoff for pycbc_make_skymap. Changes: -Reverted changes to pycbc/io/live.py -Made sure --f-upper option in pycbc_make_skymap is optional -Passing upper frequency limit to BAYSTAR via coinc_result
* Fix exception handling in inference data utils module * Simplify
* make it easier to just get the losc strain channel * less inobvious hard coding in pycbc make skymap * fixes
* Single-trigger handling - calculate single stat in general case, p_astro calculation
* add in codes and workflow to parallelize optimal_snr
* speed up fit trigs * faster there too * minor cleanup * add more comments * one more comment * minor spacing/comments changes * minor tweaks to comments and verbose output Co-authored-by: Thomas Dent <thomas.dent@usc.es>
* Prep for release
…3421) * Added ability to accept multiple frame files to pycbc_make_skymap. * Adjusted segment checks to take multiple frames into condieration. * Added option to change the sample-rate of the data. Changes: -Fixed a bug where the custom frame-files were not correctly passed to pycbc_single_template -Added option to choose the sample-rate of the input data.
Changes: -Changed the error-codes on sys.exit to 1 instead of 0. -Fixed a bug, where the wrong times were printed in error messages. Co-authored-by: Marlin Schaefer <marlin.schaefer@condor3.atlas.local>
* Change default gzip compression level from 9 to 6 in mergetrigs * Expose compression as CLI option Co-authored-by: Tito Dal Canton <dalcanton@lal.in2p3.fr>
* Remove Singularity constructs from docker container * Fix spelling error in Dockerfile * Add environment variables to Dockerfile * Do not set PATH or LAL_DATA_PATH in /etc/profile/pycbc.[c]sh
* testing single template marginalization * Create single_template_p.py single_template model with ability to marginalize over polarization angle * incorporated the necessary changes into it single_template_p added num_sample which when None, will require 'polarization' as a parameter. If num_sample has some int value, an array is created, over which marginalization takes place. If num_sample has int value, and polarization is given as a param, then array is still created and marginalization is done. * polarization angle marginalization in single_template added num_sample which when None, will require 'polarization' as a parameter. If num_sample has some int value, an array is created, over which marginalization takes place. If num_sample has int value, and polarization is given as a param, marginalization has higher precedence. * Delete single_template_p.py * Delete single_template_pol_marginalization test.ipynb * code quality updates
* register the new function * code climate
… fixed sample distribution (#3385) * recommit * new distribution * cc * fix * use dummy logp
* fixes to spa for python3 * fixes
* add alternate walk * small cc fixes
* Fix some bugs arising from early-warning test * Line too long
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
Updating fork