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

Add IMU and Gyro Drift Rates and Accelerometer Biases #1030

Open
wants to merge 47 commits into
base: Orbiter2016
Choose a base branch
from

Conversation

n7275
Copy link
Member

@n7275 n7275 commented Jul 17, 2023

This is very much WIP. Adding so it doesn't get lost.

@n7275
Copy link
Member Author

n7275 commented Aug 6, 2023

IMU drift compensation from the CMC, showing NBDX,Y,Z ADIAX,Y,Z and ADSRAX,Y,Z combined. CMC is compensating for NBD before, and after an 1800 ft/sec SPS burn, and compensating for all nine drift rates during the burn.
image

All three axes have been tested separately to confirm sign and handedness in the CM/CMC.

on to the LM...

@n7275
Copy link
Member Author

n7275 commented Mar 9, 2024

Testing State vector for Apollo 11 Launch

image
image
image

@n7275 n7275 self-assigned this May 16, 2024
@n7275 n7275 marked this pull request as ready for review May 16, 2024 03:15
@rcflyinghokie
Copy link
Collaborator

rcflyinghokie commented Jun 5, 2024

#1247 can you add this to your branch to prevent merge conflicts with OO?

Just update the LM padcount for Apollo 10 and add those two addresses zeroed, think it makes it 200 for yours

Copy link
Collaborator

@rcflyinghokie rcflyinghokie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imu.cpp the "scanf" need to be "sscanf"

@n7275 n7275 requested a review from rcflyinghokie June 6, 2024 00:06
@rcflyinghokie
Copy link
Collaborator

just need to resolve the merge conflict on the Apollo 10 LM pads that you got from pulling the main branch

@n7275
Copy link
Member Author

n7275 commented Jun 8, 2024

just need to resolve the merge conflict on the Apollo 10 LM pads that you got from pulling the main branch

Should be fixed now.

ThatRedMelon
ThatRedMelon previously approved these changes Jun 16, 2024
Copy link
Contributor

@ThatRedMelon ThatRedMelon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flown whole Apollo 11 flight without issues

Missions/ProjectApollo/Apollo 10.cfg Outdated Show resolved Hide resolved
Missions/ProjectApollo/Apollo 11.cfg Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CMC padload here reverses the changes that were done for switching from Comanche 55 to 67.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this may have been the result of a bad merge-conflict resolution on my part.

Missions/ProjectApollo/Apollo 12.cfg Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants