Add 10s timeout and restart multimon if rtl_fm crashes & fix pytest and codingstyle-failures #64
Conversation
|
@Schrolli91 Pytest passe ich auch mit an, da pep8 deprecated war |
janspeller
changed the title
janspeller
force-pushed
the
feature/sdrInput_improvements
branch
from
47e2f6f
to
9363fc8
Compare
test/pytest.ini
Outdated
| # E402 # import not at top | ||
| # E501 # line too long | ||
| # pep8maxlinelength = 99 No newline at end of file | ||
| # E722 # do not use bare 'except' | ||
| # W504 # line break after binary operator | ||
| # W605 # invalid escape sequence | ||
| # flake8-max-line-length = 99 |
There was a problem hiding this comment.
Hier sollten wir mal klären, welche Ausnahmen wir davon wirklich ignorieren wollen.
Oder ob es Sinn macht, diese zuzulassen und mittelfristig zu beheben.
There was a problem hiding this comment.
Macht vermutlich sinn, die mittelfristig zu beheben, hatte die bis jetzt noch ignoriert da hier noch einige fehler anfallen, hauptsaechlich wegen den Datei-Headern.
W504 kommt in konflikt mit W503, da ist wohl irgendwas bei flake8 vehauen, oder es gibt keine "richtige" loesung.
E722 Sollten wir glaube ich moeglichst beheben, wobei der "catch all Exceptions" ja auch schon wichtig sein kann.
There was a problem hiding this comment.
"catch all" sollte schon nach Möglichkeit vermieden werden.
An den wenigen Stellen, wo es wirklich erwünscht ist kann man das auch mit einem FLAG direkt im Sourcecode abschalten: https://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html#in-line-ignoring-errors
Jo macht absolut sinn, hab ich mit rein gepackt, schauen ob alles ohne Probleme durchlaeuft |
@Schrolli91 FYI, der stdin von multimon wird neu gesetzt, weil es hier zu crashes von mutlimon kam.