Correct app_name argument #1151
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## develop #1151 +/- ##
========================================
Coverage 70.48% 70.48%
========================================
Files 79 79
Lines 7684 7684
========================================
Hits 5416 5416
Misses 2268 2268 |
felixdivo
approved these changes
Oct 27, 2021
hardbyte
pushed a commit
that referenced
this pull request
Nov 16, 2021
* Added optional byte highlighting to viewer Added optional byte highlighting that can be toggled using the shortcut key 'h' * Some black formatting changes * Final black formatting * Fixed viewer bug Fix for the following error since the introduction of #1151 / #1142 Traceback (most recent call last): File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main "__main__", mod_spec) File "/usr/lib/python3.7/runpy.py", line 85, in _run_code exec(code, run_globals) File "/home/pi/python-can/can/viewer.py", line 582, in <module> main() File "/home/pi/python-can/can/viewer.py", line 573, in main bus = _create_bus(parsed_args, **additional_config, app_name="python-can viewer") File "/home/pi/python-can/can/logger.py", line 105, in _create_bus if parsed_args.app_name: AttributeError: 'Namespace' object has no attribute 'app_name' * Update scripts.rst update viewer documentations * Update scripts.rst * Update scripts.rst arrange keyboard shortcuts into bulleted list * Update scripts.rst * Update can/viewer.py Co-authored-by: Felix Divo <[email protected]> * Better description. Add extra length to other lines in the help text so the border lines up. * Update test_viewer.py Add test coverage for some of the new byte highlighting functions. * Use _create_base_argument_parser in viewer.py Change standard arguments definition to be defined using logger._create_base_argument_parser instead of manually defining them in this module. Co-authored-by: Felix Divo <[email protected]>
cowo78
pushed a commit
to cowo78/python-can
that referenced
this pull request
Dec 9, 2021
* Added optional byte highlighting to viewer Added optional byte highlighting that can be toggled using the shortcut key 'h' * Some black formatting changes * Final black formatting * Fixed viewer bug Fix for the following error since the introduction of hardbyte#1151 / hardbyte#1142 Traceback (most recent call last): File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main "__main__", mod_spec) File "/usr/lib/python3.7/runpy.py", line 85, in _run_code exec(code, run_globals) File "/home/pi/python-can/can/viewer.py", line 582, in <module> main() File "/home/pi/python-can/can/viewer.py", line 573, in main bus = _create_bus(parsed_args, **additional_config, app_name="python-can viewer") File "/home/pi/python-can/can/logger.py", line 105, in _create_bus if parsed_args.app_name: AttributeError: 'Namespace' object has no attribute 'app_name' * Update scripts.rst update viewer documentations * Update scripts.rst * Update scripts.rst arrange keyboard shortcuts into bulleted list * Update scripts.rst * Update can/viewer.py Co-authored-by: Felix Divo <[email protected]> * Better description. Add extra length to other lines in the help text so the border lines up. * Update test_viewer.py Add test coverage for some of the new byte highlighting functions. * Use _create_base_argument_parser in viewer.py Change standard arguments definition to be defined using logger._create_base_argument_parser instead of manually defining them in this module. Co-authored-by: Felix Divo <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.