Skip to content

Conversation

@DusKing1
Copy link
Contributor

@DusKing1 DusKing1 commented Jul 3, 2020

Doc typo fix.

DusKing1 added 2 commits July 3, 2020 11:42
Clearly 0 to 6 has 7 numbers
According to the actual CLI command and conversion diagram, the number here should be 27.
@asizon
Copy link
Member

asizon commented Jul 3, 2020

@haslinghuis
Copy link
Member

haslinghuis commented Jul 3, 2020

Also good catch @asizon but I think the number should be 28. The last 4 LED aren't used in the example. But there is a second 28 Led example in the text. So not sure how to name this?

@asizon
Copy link
Member

asizon commented Jul 3, 2020

True @haslinghuis , seems that 28 Leds example is designed for Alien Spider AQ50D PRO 250mm frame. Maybe we need to fix 12 Led example removing info from 12 to 28 leds and make a separate example for this default 28 Leds example

@haslinghuis
Copy link
Member

Or label the first 12 LEDs example as Default config with 28 LEDs and rename the second Exmple 28 LED config as Alien Spider 28 LED configuration .

@mikeller mikeller merged commit 262a2e1 into betaflight:master Jul 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants