Skip to content

Import gwpy.timeseries explicitly

John Douglas Veitch requested to merge john-veitch/lalsuite:lalinf_gwpy_timeseries into master

Description

@aaron.zimmerman found the error

Read state vector for H1 data between 1260287933.0 and 1260287966.0
Traceback (most recent call last):
  File "/cvmfs/oasis.opensciencegrid.org/ligo/sw/conda/envs/igwn-py37-20200331/bin/lalinference_pipe", line 482, in <module>
    dag=pipe_utils.LALInferencePipelineDAG(this_cp)
  File "/cvmfs/oasis.opensciencegrid.org/ligo/sw/conda/envs/igwn-py37-20200331/lib/python3.7/site-packages/lalinference/lalinference_pipe_utils.py", line 842, in __init__
    self.add_science_segments()
  File "/cvmfs/oasis.opensciencegrid.org/ligo/sw/conda/envs/igwn-py37-20200331/lib/python3.7/site-packages/lalinference/lalinference_pipe_utils.py", line 1609, in add_science_segments
    ast.literal_eval(self.config.get('statevector', 'bits')), start, end
  File "/cvmfs/oasis.opensciencegrid.org/ligo/sw/conda/envs/igwn-py37-20200331/lib/python3.7/site-packages/lalinference/lalinference_pipe_utils.py", line 62, in findSegmentsToAnalyze
    state = gwpy.timeseries.StateVector.read(
AttributeError: module 'gwpy' has no attribute 'timeseries'

This patch should solve it by importing gwpy.timeseries explicitly.

CC @duncanmmacleod

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

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Please provide details on any reviews related to this change and and the associated reviewers.

Merge request reports