oss-slu / oss-slu/PilotDataSynchronization

Add unit tests for flight event labeling

Open
#174 1 comment 0 reactions 1 assignee View on GitHub

@gaoharimran29-glitch is already working on this.

Since Sep 7, 2026.

good first issue python testing
Dominant language
Rust
Stars
4
Forks
8
Avg merge
5d 17h
Merged PRs (30d)
2

Description

Description

The project uses rules in label_generator.py to assign flight-event labels. These rules do not currently have automated unit tests.

Create tests to verify that representative telemetry values generate the expected labels.

Tasks

  • Set up pytest for the inference folder.

  • Create a test file for FlightEventLabeler.

  • Add at least one test for each event:
    TAXI
    TAKEOFF
    CRUISE
    APPROACH
    LANDING
    TURN_LEFT
    TURN_RIGHT
    HIGH_SPEED
    LOW_SPEED
    HIGH_ALTITUDE
    LOW_ALTITUDE
    HIGH_G_FORCE
    NORMAL_FLIGHT

  • Test heading wraparound, such as 359° to 1°.

  • Test values at important boundaries.

  • Add instructions for running the tests.

Acceptance Criteria

  • All 13 expected event labels are tested.
  • Heading wraparound is tested.
  • At least three threshold-boundary cases are tested.
  • Tests run with one documented command.
  • All tests pass locally.
  • Existing production code is not changed unless a bug is found.
  • Any discovered bug is documented separately.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.