Skip to content

Update LM radar self test behavior to better reflect procedures#1468

Open
rcflyinghokie wants to merge 11 commits intoorbiternassp:Orbiter2016from
rcflyinghokie:LMRadarSelfTest
Open

Update LM radar self test behavior to better reflect procedures#1468
rcflyinghokie wants to merge 11 commits intoorbiternassp:Orbiter2016from
rcflyinghokie:LMRadarSelfTest

Conversation

@rcflyinghokie
Copy link
Collaborator

@rcflyinghokie rcflyinghokie commented Jun 27, 2025

  • Adds cos 15 multiplier to tape meter slant range in hover position
  • Adds mission specific self test values
  • Add multiplier to LM-3 to display what was seen in flight on the tape meter (workaround until the actual source is found)

While this PR doesn't address "why" some of these were seen, it adds the correct behavior and would make the procedures properly align rather than using hardcoded Apollo 11 values for every LM. I would like to see it merged with the caveat that if/when better self test/tape meter information comes about we add it.

@rcflyinghokie rcflyinghokie marked this pull request as draft July 2, 2025 13:21
@rcflyinghokie rcflyinghokie marked this pull request as ready for review July 10, 2025 14:18
@rcflyinghokie
Copy link
Collaborator Author

Moving to draft for now until we can implement wiring research

@rcflyinghokie rcflyinghokie marked this pull request as draft July 19, 2025 15:31
@rcflyinghokie rcflyinghokie marked this pull request as ready for review February 14, 2026 16:40
@rcflyinghokie
Copy link
Collaborator Author

Happy with this as is until we figure the "whys" of all of these differences. As implemented here, this generates expected self test values per mission instead of a single hard coded value

@rcflyinghokie rcflyinghokie requested a review from indy91 February 18, 2026 18:05
@rcflyinghokie rcflyinghokie marked this pull request as draft February 19, 2026 16:19
@rcflyinghokie rcflyinghokie marked this pull request as ready for review February 20, 2026 16:14
@rcflyinghokie rcflyinghokie added the Close Before NASSP 9 This PR should be merged or closed before switching to NASSP 9 label Feb 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Close Before NASSP 9 This PR should be merged or closed before switching to NASSP 9

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant