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

Better Error Message for BBH Headon ID #6240

Open
AlexCarpenter46 opened this issue Aug 22, 2024 · 1 comment
Open

Better Error Message for BBH Headon ID #6240

AlexCarpenter46 opened this issue Aug 22, 2024 · 1 comment
Labels
cli/pybindings Command line interface & Python bindings

Comments

@AlexCarpenter46
Copy link
Contributor

Bug reports:

Expected behavior:

If I specify

spectre bbh generate-id -q 1.0 --chi-A 0 0 0 --chi-B 0 0 0 -e 0 -w 0 -o ID -N 1

I would expect an error like "Cannot use omega = 0 with e = 0"

Current behavior:

Instead this is what it currently prints which is not very helpful
"Failed to find a separation that gives the desired orbital angular velocity. Error: Maximum number of function evaluations has been exceeded."

@AlexCarpenter46 AlexCarpenter46 added the cli/pybindings Command line interface & Python bindings label Aug 22, 2024
@nilsdeppe
Copy link
Member

Oh, we probably need to not try to use PN to get a guess in this case... Basically, if Omega=0 then I think the PN guess can't work because PN assumes orbits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli/pybindings Command line interface & Python bindings
Projects
None yet
Development

No branches or pull requests

2 participants