Skip to content

Conversation

@ZacharyLowell
Copy link

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.

katzfey added 30 commits June 15, 2021 15:42
…atic data structures. Fixed a bad

bug in drv_hrt that was causing some clients to get times without the dsp offset and others with it. This
would cause px4 to "go dark" and need to be reset. Also cleaned up some documentation.
…o that the DSP profiler

prints them. Tweaking DSP thread priorites. Optimizing IMU and Barometer processing. Disable
UART ESC feedback.
@ZacharyLowell ZacharyLowell requested a review from katzfey May 5, 2022 21:00
@codecov-commenter
Copy link

codecov-commenter commented May 5, 2022

Codecov Report

Merging #12 (e17a6bf) into voxl-dev (3162783) will decrease coverage by 15.94%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           voxl-dev      #12       +/-   ##
=============================================
- Coverage     50.30%   34.36%   -15.95%     
=============================================
  Files           722      670       -52     
  Lines         56794    53090     -3704     
=============================================
- Hits          28570    18244    -10326     
- Misses        28224    34846     +6622     
Flag Coverage Δ
mavsdk 34.36% <ø> (-0.04%) ⬇️
unittests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...forms/common/include/px4_platform_common/defines.h 100.00% <ø> (ø)
src/lib/parameters/parameters.cpp 55.73% <ø> (-13.34%) ⬇️
src/lib/bezier/BezierN.cpp 0.00% <0.00%> (-100.00%) ⬇️
src/modules/navigator/land.h 0.00% <0.00%> (-100.00%) ⬇️
src/lib/bezier/BezierQuad.hpp 0.00% <0.00%> (-100.00%) ⬇️
src/lib/controllib/BlockP.hpp 0.00% <0.00%> (-100.00%) ⬇️
src/lib/controllib/BlockPD.hpp 0.00% <0.00%> (-100.00%) ⬇️
src/lib/controllib/BlockPI.hpp 0.00% <0.00%> (-100.00%) ⬇️
src/modules/navigator/loiter.h 0.00% <0.00%> (-100.00%) ⬇️
src/lib/controllib/BlockPID.hpp 0.00% <0.00%> (-100.00%) ⬇️
... and 301 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update afabfbb...e17a6bf. Read the comment docs.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants