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

Modalai 1.11 #1

Open
wants to merge 100 commits into
base: master
Choose a base branch
from
Open

Modalai 1.11 #1

wants to merge 100 commits into from

Conversation

danielhonies
Copy link
Member

Please use PX4 Discuss or Slack to align on pull requests if necessary. You can then open draft pull requests to get early feedback.

Describe problem solved by this pull request
A clear and concise description of the problem this proposed change will solve.
E.g. For this use case I ran into...

Describe your solution
A clear and concise description of what you have implemented.

Describe possible alternatives
A clear and concise description of alternative solutions or features you've considered.

Test data / coverage
How was it tested? What cases were covered? Logs uploaded to https://review.px4.io/ and screenshots of the important plot parts.

Additional context
Add any other related context or media.

modaltb and others added 30 commits August 28, 2020 16:28
…15348)

* X7Pro adds CAN driver
* UAVCAN timer selection moved to default.cmake
* Modify some details about @CUAVcaijie UAVCAN timer selection moved to default.cmake
* Put some timer parameters to micro_hal.h from board_config.h. Fix all h7 boards

Co-authored-by: honglang <[email protected]>
 - single register output is in 2's complement
All the other distance sensors have their default rotation to downwards
facing as well
It loads the battery parameters but then overwrites them
with hardcoded values and it breaks the ModuleParams
parent/child hierarchy. Both is undesired.
The subscription to parameter updates has to get
copied otherwise the change detection will not get
reset for next time.
This was not a problem before because battery.h
included the adc driver and implicitly snprintf
was defined through there.
 - auto disarm when locked down was added in PX4#14766 to prevent user confusion in regular usage, but also breaks HITL where lockdown is enabled for safety
 - fixes PX4#15686
…SB on new carrier boards)

- add console builds (cubepilot_cubeorange_console) for hardware test rack and developers
 - BOARD_DSHOT_MOTOR_ASSIGNMENT isn't needed if there's no remapping
I observed stack overflows when executing `uavcan params list`, so the
stack size probably needs to be increased.

Signed-off-by: Alex Mikhalev <[email protected]>
Regression from d2254c2, I overlooked that they were defined in
reverse order.
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

Successfully merging this pull request may close these issues.