BUG: Remove duplicate controller process

[zuorenchen]

Pull request type

• Code changes (bugfix, features)

Checklist

• Lint (black rocketpy/ tests/) has passed locally
• All tests (pytest tests -m slow --runslow) have passed locally
• CHANGELOG.md has been updated (if relevant)

Current behavior

#948 BUG: Controller processed twice in flight.py

New behavior

1. Remove duplicate process
2. Fix the condition for sensor measurement. Should check if the sensor exists in the time_node instead of flight.sensor

Breaking change

• No

Additional information

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.05%. Comparing base (9cf3dd4) to head (b23ab58).
⚠️ Report is 50 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #949      +/-   ##
===========================================
+ Coverage    80.27%   81.05%   +0.78%     
===========================================
  Files          104      107       +3     
  Lines        12769    13903    +1134     
===========================================
+ Hits         10250    11269    +1019     
- Misses        2519     2634     +115     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[MateusStano]

Wow, great catch!