Allow time domain approximants

Closes #308 (closed)

This changes the lalsimulation function being called and enforces frequency bounds on the waveforms returned.

I noticed there's a lot of duplicate code in the source functions. I'll make a separate issue to rationalise this.

@michael.puerrer can you check this?

UPDATE: I got carried away and made all the lal source functions call a single base function. This is likely a more maintainable solution.

added 2 commits

• 59869044 - test source functions
• ce56c551 - set up a base lal FD waveform source function

Compare with previous version

changed the description

added 1 commit

• 8834bae2 - make source tests python2 compatible

Compare with previous version

added 1 commit

• 0f8a09c2 - use different methods for time and frequency domain

Compare with previous version

added 9 commits

• 0f8a09c2...3d5ea9dc - 8 commits from branch master
• d74d6da6 - Merge branch 'master' into 'allow_time_domain_approximants'

Compare with previous version

added 2 commits

• 04c1fbce - add some more source tests
• a5013b9f - Merge branch 'allow_time_domain_approximants' of git.ligo.org:lscsoft/bilby...

Compare with previous version

added 1 commit

• f8725626 - fix roq path

Compare with previous version

added 1 commit

• 96bc5023 - fix test

Compare with previous version

added 1 commit

• 0bc286b0 - make aligned spins work again

Compare with previous version

added 1 commit

• c26bc406 - remove unused import

Compare with previous version

approved this merge request

approved this merge request

added 19 commits

• c26bc406...1db36052 - 17 commits from branch master
• f5c949d9 - Merge branch 'master' into allow_time_domain_approximants
• 43930e34 - make consistent with theta_jn

Compare with previous version

added 1 commit

• e84fd305 - rename function call

Compare with previous version

approved this merge request

changed milestone to %0.4.1

approved this merge request

mentioned in commit 30d9a9a1

merged

mentioned in issue #334 (closed)