Attitude field added to blackbox fields list

betaflight / betaflight-configurator

Cross platform configuration tool for the Betaflight firmware

GNU General Public License v3.0

2.57k stars 909 forks source link

Open demvlad opened 1 week ago

demvlad commented 1 week ago

Improvement for "Added flight IMU pitch, roll, yaw attitudes to blackbox log" PR Added "Attitude" field in blackbox log list confatt

netlify[bot] commented 1 week ago

Name	Link
Latest commit	28e9635040d45ddd52ab9d86682604aeefb8eea1
Latest deploy log	https://app.netlify.com/sites/origin-betaflight-app/deploys/6736359bd635730008bf5284
Deploy Preview	https://deploy-preview-4248.dev.app.betaflight.com
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

haslinghuis commented 1 week ago

Please rebase

demvlad commented 1 week ago

The work is restored after wrong rebase. Looked very simmilar during rebase, missed difference...

haslinghuis commented 1 week ago

@nerdCopter how would you approach ?

Would suggest using git rebase -i HEAD~60 and pick first commit and drop all others. And then probably git pull --rebase upstream master

nerdCopter commented 1 week ago

sometimes i tend to

git checkout myBranch
git fetch --all
git merge upstream/master
git rebase -i upstream/master

this seems to keep my ordered commits, while rebasing upon master.

when i play with this stuff, i tend to make new test-branches try the commands, then either

redo it in original branch and force-push, or
delete the original branch and rename the test-branch (but this second approach corrupts the PR's)

demvlad commented 1 week ago

The one code row changed only. Rebased master, created and push force new branch

sonarcloud[bot] commented 1 week ago

demvlad commented 1 week ago

The test result of BF, Configurator and explorer work are showed this