Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backwards integration #104

Open
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

OllieBurke
Copy link
Collaborator

Backwards integration scheme for FEW. Change inspiral keyword arguments: integrate_backwards: True to allow for trajectories in reverse.

Feed in final parameters defined at p_f, e_f, iota_f (such that p_f > p_sep) and evolve backwards for T years. Final parameters p0, e0, iota0 will be found as last point in trajectory.

Unittests are also incorporated.

Ollie Burke added 21 commits June 7, 2023 16:23
Spline interpolation was not working at all with MCMC. Instead
I am now modifying the phases and letting it properly integrate backwards.
The resultant waveform appears as if it were integrated forwards in time.

There is a small bug though, there is a slight phase shift between
the forwards integrated and backwards integrated waveform. Need
to fix this later. Just something to be aware of.
There is an annoying bug I cannot spot. The waveform seems like it
has a constant phase shift and I can't identify this right now.
@OllieBurke OllieBurke requested a review from lorenzsp October 11, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant