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

body:atm:sealevelpressure docs don't match reality #1970

Closed
Dunbaratu opened this issue Mar 3, 2017 · 0 comments
Closed

body:atm:sealevelpressure docs don't match reality #1970

Dunbaratu opened this issue Mar 3, 2017 · 0 comments
Assignees
@Dunbaratu
Labels
bug Weird outcome is probably not what the mod programmer expected. documentation Change the Sphinx user documents.
Milestone
v1.1.0

Comments

@Dunbaratu
Copy link
Member

Dunbaratu commented Mar 3, 2017 •
edited
Loading

The documentation claims that a body:atm:sealevelpressure is measured in ATM's.

I just did this test (in Galileo Planet Pack where Gael is supposed to be just like Kerbin) and got the answer that gael:atm:sealevelpressure is 101.

If it's 101 then clearly the units aren't ATM's.

I'm marking it as both bug and documentation for now since I don't know which is wrong - are the docs wrong and should change or is the code wrong and it should change.
@Dunbaratu Dunbaratu added bug Weird outcome is probably not what the mod programmer expected. documentation Change the Sphinx user documents. labels Mar 3, 2017
@Dunbaratu Dunbaratu added this to the v1.1.0 milestone Apr 20, 2017
@Dunbaratu Dunbaratu self-assigned this May 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dunbaratu Dunbaratu

Labels
bug Weird outcome is probably not what the mod programmer expected. documentation Change the Sphinx user documents.
Projects
None yet
Milestone
v1.1.0
Development

No branches or pull requests
1 participant
@Dunbaratu