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 atmo temps #9

Closed
wants to merge 26 commits into from
Closed

Fix atmo temps #9

wants to merge 26 commits into from

Conversation

R-T-B
Copy link
Collaborator

@R-T-B R-T-B commented Jul 3, 2020

This really messy branch is actually not as bad as it looks, I did most of this while learning git for the first time. Future pull requests will be cleaner.

The good news is I have tested this with several testers, and it does fix Atmospheric Temp issues by a series of reverts to code that I don't completely understand beyond "this one works, that one doesn't."

Feel free to review. It does fix Issue #3 however.

R-T-B added 26 commits June 30, 2020 20:46
don't propogate my build system.
…, but only caused bugs. Fixes prestja#5

Tested, appears to work.  Recommending merging.
Git is new to me.  This is not how you do things.
Revert my build system, don't need that in PR
This reverts commit d31af22.
This reverts commit 4ebc9f6.
This reverts commit 20105b0.
@R-T-B R-T-B changed the base branch from master to 1.9 July 3, 2020 22:25
@R-T-B
Copy link
Collaborator Author

R-T-B commented Jul 4, 2020

This pull request is a disaster and I'm redoing it. It doesn't even fix the issue right anyways.

@R-T-B R-T-B closed this Jul 4, 2020
@R-T-B R-T-B deleted the FixAtmoTemps branch July 4, 2020 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inaccurate atmospheric temperature calculations
1 participant