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

[FEA]: Add tensor_count property for ControlMessage #1876

Closed
2 tasks done
yczhang-nv opened this issue Aug 28, 2024 · 0 comments · Fixed by #2078
Closed
2 tasks done

[FEA]: Add tensor_count property for ControlMessage #1876

yczhang-nv opened this issue Aug 28, 2024 · 0 comments · Fixed by #2078
Labels
feature request New feature or request

Comments

@yczhang-nv
Copy link
Contributor

Is this a new feature, an improvement, or a change to existing functionality?

Improvement

How would you describe the priority of this feature request

Low (would be nice)

Please provide a clear description of problem this feature solves

For ControlMessage, msg.tensors().count is a common pattern, calling msg.tensors() might require a bit more cost than we think, we might want to consider a tensor_count property.

See #1803 (comment)

Describe your ideal solution

Add a tensor_count property for ControlMessage.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • I have searched the open feature requests and have found no duplicates for this feature request
@yczhang-nv yczhang-nv added the feature request New feature or request label Aug 28, 2024
@morpheus-bot-test morpheus-bot-test bot moved this from Todo to Review - Ready for Review in Morpheus Boards Dec 5, 2024
@rapids-bot rapids-bot bot closed this as completed in #2078 Dec 5, 2024
@rapids-bot rapids-bot bot closed this as completed in 32c982e Dec 5, 2024
@github-project-automation github-project-automation bot moved this from Review - Ready for Review to Done in Morpheus Boards Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant