Skip to content
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

Add the ability to attach Tensor objects and timestamps to ControlMessage #1511

Conversation

drobison00
Copy link
Contributor

Resolves #1502

Adds the ability to attach TensorMemory to ControlMessages
Improves get/set/list metadata functions
Adds the ability to attach grouped/keyed timestamps

@drobison00 drobison00 added non-breaking Non-breaking change improvement Improvement to existing functionality labels Feb 11, 2024
@drobison00 drobison00 requested a review from a team as a code owner February 11, 2024 23:13
morpheus/_lib/include/morpheus/messages/control.hpp Outdated Show resolved Hide resolved
morpheus/_lib/include/morpheus/messages/control.hpp Outdated Show resolved Hide resolved
morpheus/_lib/messages/module.cpp Outdated Show resolved Hide resolved
morpheus/_lib/messages/module.cpp Outdated Show resolved Hide resolved
morpheus/_lib/messages/module.cpp Outdated Show resolved Hide resolved
morpheus/_lib/src/messages/control.cpp Outdated Show resolved Hide resolved
@drobison00
Copy link
Contributor Author

/ok to test

@drobison00
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit aa8d42e into nv-morpheus:branch-24.03 Feb 16, 2024
9 checks passed
@drobison00 drobison00 deleted the devin_fea_1502_control_message_updates branch February 16, 2024 03:34
@mdemoret-nv mdemoret-nv changed the title ControlMessage improvements Add the ability to attach Tensor objects and timestamps to ControlMessage Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[FEA]: ControlMessage updates supporting TensorMemory references, timestamps, and PyObject caching
2 participants