Replies: 1 comment
-
|
The changes are suggested in PR #22. Sorry if it is not allow to modify the codes. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi, thank you for open-sourcing aircraft performance and the toolkit, in which I leant a lot from that.
While reading emission.py, I noticed that there seems to have a missing documentation in one function:
openap/openap/emission.py
Lines 30 to 36 in c9ee4cc
If it is not intended, would you mind adding few descriptions on the beta, theta, delta, and ratio? Thanks in advance.
Also, a suggestion for consistency:
Furthermore, may I know what constants 3149 and 1230 mean in these functions:
openap/openap/emission.py
Lines 38 to 62 in c9ee4cc
Thank you for your attention.
Best regards
Beta Was this translation helpful? Give feedback.
All reactions