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

set_rate_attitude(): ignores requested rate #2188

Open
ygratz opened this issue Nov 26, 2023 · 7 comments
Open

set_rate_attitude(): ignores requested rate #2188

ygratz opened this issue Nov 26, 2023 · 7 comments

Comments

@ygratz
Copy link

ygratz commented Nov 26, 2023

Hi,
Working with SITL, I see that as the SITL starts working it sends various messages at given rates.
For example the attitude message is sent every 250ms.

When I run:
telemetry.set_rate_attitude(1.0);

It does not have any affect on the message rate.
So is true for other "set rates".

What do I need to do to overcome this?
Thanks

@julianoes
Copy link
Collaborator

Is this PX4?

@ygratz
Copy link
Author

ygratz commented Nov 26, 2023

arducopter

@Jai-GAY
Copy link

Jai-GAY commented Nov 27, 2023

telemetry.set_rate_attitude(1.0);

1/4=0.25=250ms, does 1 = 250ms?

The example code does not have set rate calling for battery, after code change for my case it works with Pi4B. I have not verified whether the sending speed changes with different values.

I tried to change it to await drone.telemetry.set_rate_battery(2) #2hz, the number of printouts is the same which is controlled by these lines await asyncio.sleep(3) telem_battery.cancel(). the printouts did not double as what I had expected.

Not sure of anything to do with this parameter.

This may interest us.

After changing this at the Arducopter side SRx_EXTRA3,2 (500 msec), regardless of drone.telemetry.set_rate_battery(x) value set, the printouts are doubled. previously was six (3+3) printouts, now I have twelve (6+6).

Most ground stations set the desired stream rate by sending the REQUEST_DATA_STREAM MAVLink message to the vehicle instead of directly setting the parameters mentioned above

@julianoes
Copy link
Collaborator

julianoes commented Nov 27, 2023

Ok, I need to try to reproduce this with ArduCopter. Can I reproduce it with just SITL?

@Jai-GAY
Copy link

Jai-GAY commented Nov 27, 2023

Can I reproduce it with just SITL?

@ygratz , do you have any comment? for my cases, I am using Arducopter on cubeorange. I am not sure can it be reproduced using SITL.

@julianoes
Copy link
Collaborator

Is this still an issue?

@ygratz
Copy link
Author

ygratz commented May 20, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants