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

Fix minor issues in Lambert #495

Closed
Juanlu001 opened this issue Nov 15, 2018 · 3 comments

Comments

1 participant
@Juanlu001
Copy link
Member

commented Nov 15, 2018

Follow-up from #492

Several things to do:

@Juanlu001 Juanlu001 added the bug label Nov 15, 2018

@Juanlu001

This comment has been minimized.

Copy link
Member Author

commented Dec 9, 2018

The first two have been addressed in #505.

@Juanlu001 Juanlu001 added this to the 0.13 milestone Dec 27, 2018

@Juanlu001 Juanlu001 added this to Backlog in poliastro/poliastro Apr 26, 2019

@Juanlu001

This comment has been minimized.

Copy link
Member Author

commented Jun 17, 2019

@jorgepiloto now that we understand the algorithm a bit better, perhaps we can keep the "low level Lambert API" in this notebook https://docs.poliastro.space/en/latest/examples/Revisiting%20Lamberts%20problem%20in%20Python.html#Multiple-revolutions but decide whether we still want to use a generator, or at least change the (_, v_l), (_, v_r) = izzo.lambert(k, r0, r, tof, M=1)?

@Juanlu001

This comment has been minimized.

Copy link
Member Author

commented Jul 15, 2019

Done!

@Juanlu001 Juanlu001 closed this Jul 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.