Skip to content

IMRCT script modified to accommodate post-processing using PESummary

Apratim Ganguly requested to merge apratim.ganguly/lalsuite:master into master

Description

lalinference/bin/imrtgr_imr_consistency_test.py and lalinference/python/lalinference/imrtgr/result.html have been modified to incorporate the different directory structure produced by PESummary and CBCBayesPostProc and also the different parameter names in the posterior samples data file. The edited python script introduces a new command line argument (pepostproc) with options cbcbayespostproc (result) and pesummary (result). The changes are backwards compatible, so in absence of the new command line argument, the script produces result for samples produced using CBCBayesPostProc. Also, the output html file generated by CBCBayesPostProc (postplots.html) and PESummary (home.html) are different which were also taken into account.

The names of the directories where the inspiral, merger-ringdown and IMR runs are soft linked to in the IMRCT output directory are changed from lalinf_* to pe_* and this has been incorporated in both the python script and the result.html file.

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

Review Status

@nathan-johnson-mcdaniel and @aditya.vijaykumar have already reviewed the scripts.

Edited by Ghost User

Merge request reports