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.

Sign up for GitHub

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

Jump to bottom

PredictedPass does not clarify units of distance and time #121

Open
tomaslink opened this issue Aug 13, 2021 · 0 comments
Open

PredictedPass does not clarify units of distance and time #121

tomaslink opened this issue Aug 13, 2021 · 0 comments

Comments

@tomaslink
Copy link

It is desirable for users the class PredictedPass (or the whole package for that matter) to have more documentation (docstrings) about the parameters. In particular, position_ecef and velocity_ecef, we understand that they are meant to be in kilometers, but is not specified in the docstrings of the class.

orbit-predictor/orbit_predictor/predictors/pass_iterators.py

Line 281 in 561f432

class PredictedPass:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tomaslink