-
-
Notifications
You must be signed in to change notification settings - Fork 345
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
feat: Flysky Noble 4 Plus (NB4+) support #5095
base: main
Are you sure you want to change the base?
Conversation
c007856
to
d307b41
Compare
d5b505a
to
6779428
Compare
c966618
to
c952b8b
Compare
330b500
to
7ae769f
Compare
8bfe593
to
0096da0
Compare
6f96397
to
e1bbc88
Compare
To allow easier disabling
f696491
to
0333f12
Compare
Finally walked through all code generation dependencies. Need the naming of switches matched to have proper switch config generated and the test cannot pass without proper switch config. And one test case failed because it required more than 2 switches and in which nb4p did not have that much in current implementation. Haha, some tests harcoded the switch names to SA, SB, etc. So..... the other way round....... |
13ba5f5
to
711e198
Compare
@pfeerick @raphaelcoeffic Please review to see if you have any concerns on the changes in code generation scripts. |
Flysky Noble 4 Plus (NB4+) Support
Development progress:
Known hardware limitations: