-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fix bug in glue op #216
Fix bug in glue op #216
Conversation
@@ -201,7 +201,7 @@ def glue( | |||
Returns: | |||
EventSet with concatenated features. | |||
""" | |||
if len(inputs) == 1: | |||
if len(inputs) == 1 and isinstance(inputs[0], EventSetNode): |
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.
This fixes an unrelated bug which caused glue(literally anything) to return that same thing, discovered while passing glue(evset_list)
instead of glue(*evset_list)
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.
Oh nice catch! I guess... that same bug is in combine()
as well 😬
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.
Updated combine too
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.
Thanks!
sorted()
in the glue implementation gave the wrong order when inputs >= 11, since "input_10" as a string is less than "input_2". remove the sorted since I don't think it's needed, else we'll need to change it for a custom sorted that takes this into account.