[FEA]: Improve developer experience with MessageMeta
and MultiMessage
classes
#687
Closed
2 tasks done
Labels
feature request
New feature or request
Is this a new feature, an improvement, or a change to existing functionality?
Improvement
How would you describe the priority of this feature request
High
Please provide a clear description of problem this feature solves
How to correctly use the
MessageMeta
,MultiMessage
andTensorMemory
classes is a common error I see from developers when making custom pipelines. The main issue seems to stem around which values to use foroffset
,count
,mess_offset
andmess_count
when changing or creating newMultiMessage
classes.Related to #674
Describe your ideal solution
We could improve the developer experience in a few ways:
MultiMessage
object:MessageMeta
:mess_offset
andmess_count
from the incoming message'sMessageMeta
.MessageMeta
at the same time:mess_offset = 0
andmess_count = MessageMeta.count
TensorMemory
:offset
andcount
from the incoming message'sTensorMemory
.TensorMemory
at the same time:offset = 0
andcount = TensorMemory.count
MessageMeta
andMultiMessage
classes to check for common mistakesoffest
andcount
options which are impossible. If theMessageMeta
class only has 100 rows, then the following must be trueMultiMessage.offset + MultiMessage.count <= MessageMeta.count
. Otherwise you can index past the last item in the meta.Describe any alternatives you have considered
No response
Additional context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: