Skip to content

Conversation

haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Jun 6, 2024

  • refactors pid tuning tab to support minimum version 4.3
  • already anticipated removal in previous implementation - so code was already somewhat prepared for refactoring.

@haslinghuis haslinghuis added this to the 11.0 milestone Jun 6, 2024
@haslinghuis haslinghuis self-assigned this Jun 6, 2024
Copy link

netlify bot commented Jun 6, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 136ba78
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/666200ead181ad00085c4676
😎 Deploy Preview https://deploy-preview-4010.dev.app.betaflight.com
📱 Preview on mobile
Toggle QR Code...

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 haslinghuis force-pushed the pid-tuning-version branch from bf746fc to bb57cef Compare June 6, 2024 18:31
@haslinghuis haslinghuis force-pushed the pid-tuning-version branch from bb57cef to 136ba78 Compare June 6, 2024 18:33
Copy link

sonarqubecloud bot commented Jun 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@haslinghuis haslinghuis changed the title Refactor PID tuning to use API > 1.44 Increment minimal supported API to 1_44 (firmware 4.3) (3/4) Jun 6, 2024
Copy link
Member

@nerdCopter nerdCopter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minimal testing of mashup 4008,9,10 -- seems fine

@haslinghuis haslinghuis merged commit a0ef597 into betaflight:master Jun 7, 2024
@haslinghuis haslinghuis deleted the pid-tuning-version branch June 7, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants