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.

Sign up for GitHub

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

Jump to bottom

Enhancements to visualize.py: Code Refactor, Improved Readability, Typing and Use Python3.10 #14

Open

dariocazzani wants to merge 6 commits into JonathonLuiten:main from dariocazzani:main

dariocazzani commented Oct 17, 2023

What does this PR do:

Use Python3.10
Improved Imports and Typing
Variable Renaming and Commenting (don't use single letters like w or h or abbreviations --> readability > short code)
Code Refactoring and Optimization/
Bug Fixes and Error Handling
- Added a check to raise a ValueError for unsupported values of the ADDITIONAL_LINES variable.

dariocazzani added 6 commits

October 17, 2023 19:22


          Use Python3.10

0f6562f


          Setup camera is more verbose but clearer

9df9a7e


          Added comments and variable names tell the story

d9c9291


          Removed bad comments

062acee


          Small comments changes and type hints

c4cdcf8


          Refactor loss computation

7cb50d2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet