Fixing dependency on ligo.lw
All threads resolved!
All threads resolved!
Getting rid of dependence on ligolw in bayeswave_pipe. Also installs megaplot.py and all of BayesWaveUtils into environment. Adding BayesWaveUtils to the path should be unnecessary (and multiple installations of BayesWave will require multiple environments).
This adds gwpy as a dependence, so the continuous integration should be OK if we can fix the docker/container issue. EDIT: Fixed with merge 79df7902
This addresses !234 (closed)
Edited by Sophie Hourihane
Merge request reports
Activity
added bayeswave_pipe label
assigned to @tyson-littenberg, @katerina.chatziioannou, and @sophie.hourihane
- Resolved by Meg Millhouse
Hi @sophie.hourihane -- I think the read function here also needs
tablename='sim_inspiral' - Last reply by Sophie Hourihane
added 1 commit
- a2c37d36 - Hack to get gwpy updated version installed for CI
added 1 commit
- 5bbac154 - Changes to install, BWUtils installs in an env
added 17 commits
-
5bbac154...79df7902 - 4 commits from branch
lscsoft:master - 42069d0a - Changing glue dependencies to ligo.lw
- 6ecbd8ba - Updating install
- e1f157f2 - Replacing ligo.lw entirely with gwpy EventTable
- aba06114 - Now specifying what directory code should be installed to
- 1c764215 - Getting rid of dependence on ligolw and changing BayesWaveUtils install...
- 3d214cdc - Fixing typo
- e674dd31 - Deleting unused line and function
- 586d8cce - Adding gwpy to yaml file
- b9ce08fc - Explicitly passing eventTable type to gwpy.EventTable
- 7778cd8b - Adding to all calls to gwpy EventTable
- 0b5ca261 - Hack to get gwpy updated version installed for CI
- 9e09d33a - Changes to install, BWUtils installs in an env
- defdd0de - Deleting hack that installs gwpy
Toggle commit list-
5bbac154...79df7902 - 4 commits from branch
mentioned in commit 77da0cc8
mentioned in merge request !234 (closed)
Please register or sign in to reply