Skip to content
New issue

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

#

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

Already on GitHub? # 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:

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

No branches or pull requests
1 participant
@tomaslink