-
Notifications
You must be signed in to change notification settings - Fork 13
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
Use pwm esc unconditionally #829
Conversation
f0d7ffd
to
0a46375
Compare
0a46375
to
b85fec8
Compare
Since actuator_output max, min and disarmed default values have been changed for HITL (similar to real drone), the flight behavior of supported HITL models might be different, previous default HITL actuator_output max, min and disarmed values have been added to HITL configs in |
da3a5e7
to
7f913a6
Compare
src/drivers/pwm_esc/pwm_esc.cpp
Outdated
@@ -31,9 +31,9 @@ | |||
* | |||
****************************************************************************/ | |||
|
|||
/** | |||
/**const uint32_t reversible_outputs = _mixing_output.reversibleOutputs(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leftovers inside comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, that's dropped there by mistake. I will clean up
@@ -31,19 +31,13 @@ | |||
# | |||
############################################################################ | |||
|
|||
if(${PX4_BOARD_VENDOR}_${PX4_BOARD_MODEL} STREQUAL "px4_sitl") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we actually need to make any modifications to pwm_out_sim, if we are not using the component?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, so maybe i should leave it untouched ? so that the module is still in sync with upstream.
0a96618
to
61452ac
Compare
4838f0c
to
b99b7cd
Compare
b99b7cd
to
9f65415
Compare
pwm_out_sim
pwm_esc
for HITL modepwm_esc
to be used at the same with UAVCAN ESCTested on saluki v3 HIITL mode with mc, fw and vtol