MAVError is an undefined name in generator/mavparse.py #116
Comments
cclauss
pushed a commit
to cclauss/pymavlink
that referenced
this issue
Aug 31, 2017
Undefined names can cause `NameError` exceptions to be raised at runtime. It is also helpful for spotting Python 3 incompatibilities (raw_input(), unicode(), xrange(), etc.). Adding this test to our critical error search will increase runtime stability. ArduPilot#114, ArduPilot#115, and ArduPilot#116 should be fixed __before__ this PR is merged.
|
No file changes here. why ? |
|
@amilcarlucas My attempt to fix resulted in circular imports so I opened an issue in hopes that fresh eyes could find a solution. |
cclauss
pushed a commit
to cclauss/pymavlink
that referenced
this issue
Sep 7, 2017
Fixes ArduPilot#116,ArduPilot#117 In pymavlink/generator/mavparse.py from .mavgen_python import MAVError to resolve undefined name which can raise `NameError` at runtime.
peterbarker
pushed a commit
that referenced
this issue
Jan 10, 2018
Undefined names can cause `NameError` exceptions to be raised at runtime. It is also helpful for spotting Python 3 incompatibilities (raw_input(), unicode(), xrange(), etc.). Adding this test to our critical error search will increase runtime stability. #114, #115, and #116 should be fixed __before__ this PR is merged.
amilcarlucas
pushed a commit
to amilcarlucas/pymavlink
that referenced
this issue
Jan 10, 2018
Undefined names can cause `NameError` exceptions to be raised at runtime. It is also helpful for spotting Python 3 incompatibilities (raw_input(), unicode(), xrange(), etc.). Adding this test to our critical error search will increase runtime stability. ArduPilot#114, ArduPilot#115, and ArduPilot#116 should be fixed __before__ this PR is merged.
amilcarlucas
pushed a commit
to amilcarlucas/pymavlink
that referenced
this issue
Jan 10, 2018
In pymavlink/generator/mavparse.py from .mavgen_python import MAVError to resolve undefined name which can raise `NameError` at runtime. Resolve the circular imports between mavgen_python.py, mavparse.py, and mavtemplate.py by putting MAVError and MAVParseError into their own maverrors.py file. Fixes ArduPilot#116,ArduPilot#117
kd0aij
pushed a commit
to kd0aij/pymavlink
that referenced
this issue
Jun 19, 2018
Undefined names can cause `NameError` exceptions to be raised at runtime. It is also helpful for spotting Python 3 incompatibilities (raw_input(), unicode(), xrange(), etc.). Adding this test to our critical error search will increase runtime stability. ArduPilot#114, ArduPilot#115, and ArduPilot#116 should be fixed __before__ this PR is merged.
kd0aij
pushed a commit
to kd0aij/pymavlink
that referenced
this issue
Jun 19, 2018
In pymavlink/generator/mavparse.py from .mavgen_python import MAVError to resolve undefined name which can raise `NameError` at runtime. Resolve the circular imports between mavgen_python.py, mavparse.py, and mavtemplate.py by putting MAVError and MAVParseError into their own maverrors.py file. Fixes ArduPilot#116,ArduPilot#117
beforezod
pushed a commit
to beforezod/pymavlink
that referenced
this issue
Feb 26, 2019
In pymavlink/generator/mavparse.py from .mavgen_python import MAVError to resolve undefined name which can raise `NameError` at runtime. Resolve the circular imports between mavgen_python.py, mavparse.py, and mavtemplate.py by putting MAVError and MAVParseError into their own maverrors.py file. Fixes ArduPilot#116,ArduPilot#117
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
MAVErroris defined ingenerator/mavgen_python.pyand is used ingenerator/mavparse.pybut is never imported.The text was updated successfully, but these errors were encountered: