Skip to content

Fixed p1_display reset timing plot crash.#360

Merged
adamshapiro0 merged 2 commits intomasterfrom
message-bytes
Mar 24, 2025
Merged

Fixed p1_display reset timing plot crash.#360
adamshapiro0 merged 2 commits intomasterfrom
message-bytes

Conversation

@adamshapiro0
Copy link
Collaborator

Changes

  • Renamed MessageData.messages_bytes list to message_bytes to match message_index list

Fixes

  • Don't delete the message_index list during numpy conversion if return_message_index=True

@adamshapiro0 adamshapiro0 requested a review from axlan March 18, 2025 13:48
@adamshapiro0 adamshapiro0 self-assigned this Mar 18, 2025
If keep_messages is False but return_message_index is True, we do not want to
delete the index.
This is consistent with message_index.
Copy link
Collaborator

@axlan axlan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@adamshapiro0 adamshapiro0 merged commit f5d3347 into master Mar 24, 2025
14 checks passed
@adamshapiro0 adamshapiro0 deleted the message-bytes branch March 24, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants