Skip to content

Conversation

kimichenn
Copy link
Contributor

@kimichenn kimichenn commented May 27, 2025

  • Implement (fake) mapping tick when manual landing is called.
  • It does a two pass system (like the mapping integration test), and then it stays in manual landing.
  • This is done through a new isMappingDone state variable, which is set to true after mapping is done after the second pass.
  • Mapping is also updated to allow a single pass system, though after some naive testing, it doesn't seem to be performing well, so we're still sticking to the two pass system.
  • For future work, we probably should implement multithreading for the first pass (see Mapping Multithreading #254)

@kimichenn kimichenn changed the base branch from main to feat/cv-everything May 27, 2025 00:20
@kimichenn kimichenn changed the title Feat/cv everything mappingtick Mapping Integration May 30, 2025
@kimichenn kimichenn marked this pull request as ready for review May 30, 2025 00:58
@kimichenn kimichenn requested a review from AskewParity June 7, 2025 03:46
Copy link
Contributor

@AskewParity AskewParity left a comment

Choose a reason for hiding this comment

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

Looks good to me, we should still SITL test it.

@tuas-travis-ci tuas-travis-ci merged commit a904322 into feat/cv-everything Jun 7, 2025
1 of 2 checks passed
@tuas-travis-ci tuas-travis-ci deleted the feat/cv-everything-mappingtick branch June 7, 2025 04:10
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.

4 participants