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

Copter, Rover: add support for DO_SET_ROI_LOCATION #7658

Open
rmackay9 opened this issue Feb 5, 2018 · 3 comments
Open

Copter, Rover: add support for DO_SET_ROI_LOCATION #7658

rmackay9 opened this issue Feb 5, 2018 · 3 comments
Labels

Comments

@rmackay9
Copy link
Contributor

@rmackay9 rmackay9 commented Feb 5, 2018

We should add support for the new ROI messages to Copter and Rover that can be found here

  • DO_SET_ROI_NONE - cancels the ROI
  • DO_SET_ROI_WPNEXT_OFFSET
  • DO_SET_ROI_LOCATION
@drprofpatrick

This comment has been minimized.

Copy link

@drprofpatrick drprofpatrick commented Sep 4, 2019

Has this been solved yet? This would be very useful for work.

@rmackay9

This comment has been minimized.

Copy link
Contributor Author

@rmackay9 rmackay9 commented Sep 4, 2019

@drprofpatrick, no i'm afraid not. the regular DO_SET_ROI works of course though so I think it's possible to use this existing command to do the same thing. It just requires a little more work creating the mission because the user will need to input the lat/lon.

@rmackay9

This comment has been minimized.

Copy link
Contributor Author

@rmackay9 rmackay9 commented Nov 11, 2019

We've got partial support this in Copter/Rover-4.0 because we support MAV_CMD_DO_SET_ROI_LOCATION. We still don't support these variations:

  • MAV_CMD_DO_SET_ROI_WPNEXT_OFFSET
  • MAV_CMD_DO_SET_ROI_NONE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.