[FEA] Adopt newer version of Matx #317
Assignees
Labels
feature request
New feature or request
Comments
Merged
rapids-bot bot
pushed a commit
that referenced
this issue
Feb 22, 2023
* Remove usage of `tensorShape_t` which was deprecated, and later removed. * Replace usage of tensor constructor in favor of the recommended `make_tensor` helper method. * Adds more C++ unittests * RMMTensor marked as a public symbol so the C++ tests can use it * Add `cuda-nvtx` package to CI driver build, needed for matx-0.3.0 Includes changes from PR #688 fixes #317 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: #667
jjacobelli
pushed a commit
to jjacobelli/Morpheus
that referenced
this issue
Mar 7, 2023
* Remove usage of `tensorShape_t` which was deprecated, and later removed. * Replace usage of tensor constructor in favor of the recommended `make_tensor` helper method. * Adds more C++ unittests * RMMTensor marked as a public symbol so the C++ tests can use it * Add `cuda-nvtx` package to CI driver build, needed for matx-0.3.0 Includes changes from PR nv-morpheus#688 fixes nv-morpheus#317 Authors: - David Gardner (https://github.com/dagardner-nv) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: nv-morpheus#667
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
We are on the initial release v0.1.0, while the most recent release is v0.3.0
In the past we were blocked on upgrading due to dependency issues, but that is no longer the case.
Describe the solution you'd like
Adopt v0.3.0 using newer factory methods like
make_tensorrather than thetensor_tconstructors.Additional context
I took a pass at it, and realized there was a bit more involved than I had time for at the time.
The text was updated successfully, but these errors were encountered: