Skip to content
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

CLI command for align mag doesn't stick #109

Open
AlexC176 opened this issue Feb 10, 2021 · 2 comments
Open

CLI command for align mag doesn't stick #109

AlexC176 opened this issue Feb 10, 2021 · 2 comments

Comments

@AlexC176
Copy link

AlexC176 commented Feb 10, 2021

SPRacing F3 rc1.1.1.0 sept 23rd 2020, align mag command doesn't take effect, I had to manually move the mag within the case to the correct alignment.

Looking at serial_cli.c the line align mag entry is commented out. Was previously raised as an issue, comment was merged into development branch. Looks like change wasn't implemented.

{ "align_mag", VAR_UINT8 | MASTER_VALUE | MODE_LOOKUP, &masterConfig.sensorAlignmentConfig.mag_align, .config.lookup = { TABLE_ALIGNMENT } },

@AlexC176
Copy link
Author

Previous closed issue was #17 (comment)

@raul-ortega
Copy link
Owner

align_mag is unommented in master and development versions since quite time ago.
Please, could you be more explicit about the issue you experienced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants