Skip to content

msg: VehicleCommandAck: bump queue length from 4 to 8#26217

Merged
julianoes merged 1 commit intomainfrom
pr-command_ack_queue
Jan 6, 2026
Merged

msg: VehicleCommandAck: bump queue length from 4 to 8#26217
julianoes merged 1 commit intomainfrom
pr-command_ack_queue

Conversation

@dakejahl
Copy link
Copy Markdown
Contributor

@dakejahl dakejahl commented Jan 5, 2026

Increase VehicleCommandAck ORB_QUEUE_LENGTH to match VehicleCommand
https://discord.com/channels/1022170275984457759/1022185721450213396/1456709818474828079
image

@julianoes
Copy link
Copy Markdown
Contributor

I have a dejavu that I wanted to do this as well at some point.

@julianoes
Copy link
Copy Markdown
Contributor

Ignoring unrelated CI.

@julianoes julianoes merged commit 6bc24c8 into main Jan 6, 2026
74 of 76 checks passed
@julianoes julianoes deleted the pr-command_ack_queue branch January 6, 2026 01:07
@DonLakeFlyer
Copy link
Copy Markdown
Contributor

So with equal side queue on both ends does this mean I'll get command failures when the queue is full as opposed to lost acks? If the command fails for this case is there anyway to know it's because of this queue full on the QGC side of things?

@dakejahl
Copy link
Copy Markdown
Contributor Author

So with equal side queue on both ends does this mean I'll get command failures when the queue is full as opposed to lost acks? If the command fails for this case is there anyway to know it's because of this queue full on the QGC side of things?

I don't think publication of the command fails. I believe uORB will always publish and the queue will quietly overflow. Ideally we would have a mechanism to prevent that, and have publish() fail if the queue is full.

@DonLakeFlyer
Copy link
Copy Markdown
Contributor

Ideally we would have a mechanism to prevent that, and have publish() fail if the queue is full.

yeah, that's kinda nasty behavior

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.

3 participants