-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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: fix multimodal example #2178
Conversation
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
Codecov Report
@@ Coverage Diff @@
## master #2178 +/- ##
==========================================
+ Coverage 90.41% 90.51% +0.10%
==========================================
Files 208 208
Lines 11243 11243
==========================================
+ Hits 10165 10177 +12
+ Misses 1078 1066 -12
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -22,6 +24,7 @@ def craft(self, tags): | |||
class BiSegmenter(Segmenter): | |||
"""Segmenter for multimodal example.""" | |||
|
|||
@single_multi_input(num_data=2, flatten_output=False) |
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 setting increases the learning time for developers. They not only need to learn which exec to choose, but also determine how to decorate the method.
Is it possible to maintain the previous code and behavior, and we only add these decorators when needed?
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.
I would love to do so, but the cost for this would mean having an extra set of Drivers, (with an explosion of Drivers).
One thing I wanted to do was to see if we can add the decorators automatically.
But yes, it is a problem from this point of view.
What can we do about this @nan-wang ?
96549e8
to
cf53f44
Compare
Shall we merge this? |
reject. as |
How will it be handled then? |
No description provided.