-
Notifications
You must be signed in to change notification settings - Fork 991
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
Unexpected behavior in offboard mode #273
Comments
what is the difference to the python script from the viewpoint of the autopilot --> Do both approaches use the exact same set of mavlink messages? |
Different encoders — bug can't be here! // this is needed for now to avoid a throttle cut...
sp.vx = 0.0;
sp.vy = 0.0;
sp.vz = 0.0; So you may try edit type_mask: uint16_t ignore_all_except_xyzy = (1 << 11) | (7 << 6); // ignore velocity vector: | (7 << 3); |
@vooon Yes, changing the mask works ! Maybe you should make this change permanent in mavros (Not sure if this would effect APM) ? I will test the velocity set point next (the velocity setpoint in the current firmware doesn't work, so I will use an older PX4 firmware) |
We should fix this in PX4 |
ah I was to fast...
needed as well? Probably you were sending 0 for the velocities anyway? I am confused why setting them again to 0 as per @voon 's comment should change something |
In mavros velocity vector is zero (all ignored data is zero), but before it ignored. |
And i think that it is bug, so this qirk may be removed after fixing PX4/PX4-Autopilot#2036 . |
api#L47 is definitely a hack. At the time I wrote this, ignoring the velocity components didn't help. Are you all able to command velocity setpoints? See PX4/Firmware/#1856 |
@thomasgubler as @vooon said, the mavros sets the velocity vector to zero before sending it, so I just changed the mask |
@BlackCoder could you lend a hand for debugging the velocity issue? |
@thomasgubler yes of course, just let me how do you want to approach it. |
@BlackCoder @aerialhedgehog Have you seen this branch / PR? PX4/PX4-Autopilot#1983 |
@LorenzMeier , no I didn't see it before. I can test it in the next couple of days. What is the expected behaviour resulting from this PR? |
* master: plugin: param fix #276: add check before reset request downcounter. libmavconn #269: add seq number to debug gcs bridge fix #277: add link diagnostics vim: Add YouCompleteMe configuration file plugin: setpoint_position #273: add quirk for PX4. readme: fir glossary misprint readme: add notes about catkin tool 0.11.1 prepare release 0.11.1 scripts #262: update mavwp scripts #262: mavsetp, new module mavros.setpoint mavftpfuse #129: done! mavftpfuse #129: cache file attrs mavftpfuse #129: initial import
@LorenzMeier this is new to me too, will check it out |
@LorenzMeier it is fixed? |
@LorenzMeier Can I use the velocity control command with master firmware now? |
This is fixed in that it won't cut throttle. |
Support dialects in mavgraph.py
I am trying to perform offboard navigation using mavros following this procedure :
script used to send setpoint: https://raw.githubusercontent.com/kuri-kustar/indoor_uav_control/master/scripts/setpoint_demo_2.py
The plot shows how immediately after changing to offboard mode, the motors disengage/go back to trim values.
a similar procedure using the uar_interface works fine: https://github.com/kuri-kustar/c_uart_interface_example/tree/object
The text was updated successfully, but these errors were encountered: