Skip to content

DepthAI Constants Refactoring #92

@maxwellmlin

Description

@maxwellmlin

Some constants tuning here that we should consider refactoring (either now or in a future PR):

  • We tuned the xyz coords of torpedo_banner for our torpedo banner navigation algorithm. I think we should move these constants to the CV config file and move the multiplication to calculate_relative_pose.
  • We negate yaw here. We should negate earlier (where we call publish_prediction).

Given that this PR needs to be merged and this is a refactoring issue (not correctness), I think it's OK to push this issue to a future PR.

Originally posted by @maxwellmlin in #82 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions