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

added vtx_smartaudio_early_akk_workaround option #6458

Merged
merged 7 commits into from Feb 5, 2021

Conversation

RomanLut
Copy link
Contributor

Solution for #6456

…txSAGetPowerIndex() is trying to decode power index from mW
…fixed: vtxSAGetPowerIndex() is trying to decode power index from mW"

This reverts commit 71a4e32.
@RomanLut
Copy link
Contributor Author

I would like to ask someone to run src/utils/update_cli_docs.py because I have problems running it on my dev env.

@DzikuVx DzikuVx added this to the 2.7 milestone Dec 29, 2020
@DzikuVx
Copy link
Member

DzikuVx commented Dec 29, 2020

@RomanLut please update docs with python3 src/utils/update_cli_docs.py

@RomanLut
Copy link
Contributor Author

Unfortunately I have problem running this script under MacOS:

Traceback (most recent call last):
File "src/utils/update_cli_docs.py", line 55, in
md_table_lines = generate_md_table_from_yaml(settings_yaml)
File "src/utils/update_cli_docs.py", line 37, in generate_md_table_from_yaml
md_table_lines.append("| {} | {} | {} |\n".format(param[0], param[1]['default'], param[1]['description']))
UnicodeEncodeError: 'ascii' codec can't encode character u'\xb0' in position 62: ordinal not in range(128)

There is indeed degree symbol u'\xb0' in the settings.yaml.

I would kindly ask someone runt this script or help me.

@DzikuVx
Copy link
Member

DzikuVx commented Jan 7, 2021

@RomanLut please try now after merging master to your local branch.

@RomanLut
Copy link
Contributor Author

RomanLut commented Jan 9, 2021

  1. Updated local master from master
  2. installed pyyaml
  3. run script
    now I get a lot of errors:
    Mac-Admin:inav-master admin$ python3 src/utils/update_cli_docs.py
    Setting "gyro_notch_hz" has no description or default value specified
    Setting "gyro_notch_cutoff" has no description or default value specified
    Setting "gyro_to_use" has no description or default value specified
    Setting "acc_notch_hz" has no description or default value specified
    Setting "acc_notch_cutoff" has no description or default value specified
    Setting "opflow_scale" has no description or default value specified

no files are changed after running script.

@RomanLut
Copy link
Contributor Author

RomanLut commented Jan 9, 2021

Probably I have to open separate issue about running update_cli_docs.py on macOs.
For now, I updated settings.md manually.
I hope it will work.

src/main/io/vtx_control.h Show resolved Hide resolved
added smartAudioEarlyAkkWorkaroundEnable initialization
@DzikuVx
Copy link
Member

DzikuVx commented Feb 5, 2021

Thanks a lot, merging!

@DzikuVx DzikuVx added the Release Notes Add this when a PR needs to be mentioned in the release notes label Feb 5, 2021
@DzikuVx DzikuVx merged commit c35b14e into iNavFlight:master Feb 5, 2021
@RomanLut RomanLut mentioned this pull request Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants