Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{Sponsored by Intelliterra} OPEN_DRONE_ID_SYSTEM, OPEN_DRONE_ID_SELF_ID, and OPEN_DRONE_ID_OPERATOR_ID values getting cached within PX4 #1

Merged
merged 8 commits into from
Jul 7, 2023

Conversation

jnomikos
Copy link

@jnomikos jnomikos commented Jul 6, 2023

Sponsored by Intelliterra

Solved Problem

OPEN_DRONE_ID_SYSTEM, OPEN_DRONE_ID_BASIC_ID, and OPEN_DRONE_ID_OPERATOR_ID values were not being cached within PX4 when received from QGC.

Note: I have not tested OPEN_DRONE_ID_SYSTEM yet as I have not been able to arm.

OPEN_DRONE_ID_OPERATOR_ID is sending data through the cube ID, and it is showing up on the Drone Scanner app. The drone scanner app gives the warning "Invalid Data", but is displaying the data correctly.

OPEN_DRONE_ID_SELF_ID is showing up on the Drone Scanner app on "Operator Description" and is showing character numbers rather than actual characters. Might be a bug on that app itself.

image
image

Context

PX4#21777
PX4#21647

Copy link
Owner

@dirksavage88 dirksavage88 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 so far, just minor stuff that needs a closer look

@dirksavage88 dirksavage88 merged commit eaa6937 into dirksavage88:remote_id_arm Jul 7, 2023
@jnomikos jnomikos changed the title OPEN_DRONE_ID_SYSTEM, OPEN_DRONE_ID_SELF_ID, and OPEN_DRONE_ID_OPERATOR_ID values getting cached within PX4 {Sponsored by Intelliterra} OPEN_DRONE_ID_SYSTEM, OPEN_DRONE_ID_SELF_ID, and OPEN_DRONE_ID_OPERATOR_ID values getting cached within PX4 Aug 3, 2023
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.

2 participants