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

Add MAV_CMD_CONDITION_YAW #8228

Closed
LorenzMeier opened this issue Nov 2, 2017 · 12 comments
Closed

Add MAV_CMD_CONDITION_YAW #8228

LorenzMeier opened this issue Nov 2, 2017 · 12 comments
Assignees

Comments

@LorenzMeier
Copy link
Member

We need this for QGC structure scan.

http://mavlink.org/messages/common#MAV_CMD_CONDITION_YAW

@DonLakeFlyer
Copy link
Contributor

Also Mavlink spec is not specific enough. Should be updated for this:
Mission Param #2 - speed during yaw change:[deg per second], 0 - use default turn rate

@DonLakeFlyer
Copy link
Contributor

DonLakeFlyer commented Nov 2, 2017

Hmm, now that I have this coded on the QGC side I realize this isn't quite what I want. Normally the vehicle points forward along the flight path (param controlled). But what I want is instead for it to point 90 degrees clockwise to the flight path as it flies the mission. Not quite sure how to do that. This keeps it pointed at the structure as it flies the polygon path.

@DonLakeFlyer
Copy link
Contributor

Example:
screen shot 2017-10-31 at 7 45 59 pm

Flies this:
screen shot 2017-10-31 at 7 18 56 pm

With the non gimbaled vehicle always pointing at the structure to keep camera in right direction.

@dagar
Copy link
Member

dagar commented Nov 2, 2017

What's the interpretation of these CONDITION commands?
Based on some of the others it sounds like MAV_CMD_CONDITION_YAW should block mission acceptance (advancement) until that yaw is achieved, but it sounds like you want to use it to command a yaw during mission?

@DonLakeFlyer
Copy link
Contributor

They do both, change to the yaw but they don't move to the next command until the yaw requested is reached. It need something which is yaw 90 degrees to the mission flight path though which this doesn't do.

@dagar
Copy link
Member

dagar commented Nov 2, 2017

We could also do it by extending MIS_YAWMODE, but I think it would be better with mavlink commands.
https://dev.px4.io/en/advanced/parameter_reference.html#mission

Thinking out loud here, but would it make sense to do it with a new ROI? That way it could be done with either a gimbal or turning the vehicle if the camera is fixed.

image

@DonLakeFlyer
Copy link
Contributor

It's only sort of an ROI. Since the flight path really determines it, just with a 90 degreee offset. A paramater won't work since you don't want to futz a param just to get a misison to work.

@dagar
Copy link
Member

dagar commented Nov 2, 2017

Yes I agree we shouldn't do it with a parameter. I meant adding a new ROI type for a particular yaw. Telling the vehicle higher level what you want (look at yaw 90 degrees) rather than specific moves to do it. Then based on the particular vehicle and configuration the firmware can do it either with the gimbal or yawing during the mission.

@DonLakeFlyer
Copy link
Contributor

DonLakeFlyer commented Nov 2, 2017

So it would be something like:
MAV_ROI_WPNEXT_YAW_OFFSET - Point toward next waypoint with offset in yaw
And then you stick Yaw degrees in param 4 or something.

@DonLakeFlyer
Copy link
Contributor

So if I used 90 degress with MAV_ROI_WPNEXT_YAW_OFFSET and flew a clockwise pattern it would always point to the structure.

@dagar
Copy link
Member

dagar commented Nov 3, 2017

That sounds good to me, but probably needs a bit more thought overall.

@PX4BuildBot
Copy link
Collaborator

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

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

No branches or pull requests

4 participants