-
Notifications
You must be signed in to change notification settings - Fork 51
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
Exporter Typing with mypy
[AARD-1765]
#1093
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing Work!
Co-authored-by: PepperLola <[email protected]>
Chorus detected one or more security issues with this pull request. See the Checks tab for more details. As a reminder, please follow the secure code review process as part of the Secure Coding Non-Negotiable requirement. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good Looking 🌹
"Fixed" 400+ type errors...
In addition to the type error fixes:
mypy
typing workflow..pyi
typing files for our compiled proto files for lsp and intellisense.GlobalManager
where the internal class instance was the only thing ever being referenced.synthesis/exporter/SynthesisFusionAddin/src/GlobalManager.py
Line 56 in 5807353
__new__
method the class was simply not working how it was originally designed.OString
needing more defined behaviour instead of revolving around the python 2object
type.synthesis/exporter/SynthesisFusionAddin/src/Types.py
Line 281 in 5807353
JointHierarchy.py
's class structure being poorly defined.mypy
pointed out but the whole structure needs to be rewritten, particularly when it comes to how the various type of nodes are currently being used, many of which I'm sure can just be removed.default_factory
s forExporterOptions
'list
defaultfield
s.ExporterOptions
field
s ofNone
where appropriate.Transition: AARD-1765
comment tags for references to individual code sections.construct_info
that would causeAttributeError
s to be thrown during exporting making debugging painful.synthesis/exporter/SynthesisFusionAddin/src/Parser/SynthesisParser/Utilities.py
Lines 54 to 59 in 5807353
JIRA Issue