Skip to content

lalinference_coherence_test.py -- logsumexp is moved from scipy.misc to scipy.special

Carl-Johan Haster requested to merge cjhaster/lalsuite:fixCoherenceTestLogSumExp into master

Description

lalinference_coherence_test.py breaks under recent SciPy versions, since after v0.19 the logsumexp function has been moved from scipy.misc to scipy.special. This MR adds a try-Except check for this which is compatible with both locations of this function.

Note that the current version of LALInference (which is installed as part of the conda packages) is broken since it calls scipy.misc, but the conda SciPy version has the function in scipy.special already.

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

no review needed (apart from the general review of the MR)

Merge request reports