-
Notifications
You must be signed in to change notification settings - Fork 423
Conversation
app/components/MessagePathSyntax/useCachedGetMessagePathDataItems.ts
Outdated
Show resolved
Hide resolved
@@ -589,8 +583,8 @@ export default class SceneBuilder implements MarkerProvider { | |||
this.collectors[topic]!.addMarker(marker as any, name); | |||
} | |||
|
|||
_consumeMarker(topic: string, message: BinaryMarker | BinaryInstancedMarker): void { | |||
const namespace = message.ns(); | |||
_consumeMarker(topic: string, message: BaseMarker): void { |
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.
There is also an _addMarker
function that is substantially identical. I think only one of these two functions should survive the 🔪
@@ -369,7 +366,7 @@ export default class RandomAccessPlayer implements Player { | |||
this._end, | |||
); | |||
|
|||
const { parsedMessages: messages, bobjects } = await this._getMessages(start, end); | |||
const { parsedMessages: messages } = await this._getMessages(start, end); |
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.
should we be renaming this parsedMessages field to lazyMessages or just messages?
@@ -469,7 +461,7 @@ export default class RandomAccessPlayer implements Player { | |||
); | |||
return undefined; | |||
} | |||
if (!topic.datatype) { | |||
if (topic.datatype === "") { |
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 wonder if this is a bad type def - it seems unlikely that someone would have an empty string, more likely that it would be missing, but I'm guessing that the type defs don't allow it to be missing.
760fdfb
to
433a07e
Compare
433a07e
to
3d50687
Compare
RewriteBinaryDataProvider is removed and logic in MemoryCacheDataProvider is updated to produce lazy messages into blocks.
3d50687
to
f78a535
Compare
Remove separate "bobjects" code paths and provide lazy messages using the existing messages code paths.