-
Notifications
You must be signed in to change notification settings - Fork 466
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
[MINOR] Python API: removal of enum output_type #2119
Conversation
5d1d1f2
to
e244d90
Compare
def code_line(self, var_name: str, unnamed_input_vars: Sequence[str], | ||
named_input_vars: Dict[str, str]) -> str: | ||
code_line = super().code_line(var_name, unnamed_input_vars, named_input_vars) | ||
return code_line | ||
|
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.
removed since we call anyways the super class method, so no need to overwrite imo
|
||
def data_type(self): | ||
return "operation_node" | ||
|
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.
can be removed, artefact from an older approach
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.
if we can remove it, lets.
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.
LGTM, if the tests pass, then feel free to merge it.
e244d90
to
ed2c983
Compare
this PR removes the redundant information about the datatype from the python DAGNodes, the datatype can be instead inferred from the instance class and the value type directly from the Java object