nv-morpheus / Morpheus

Morpheus SDK
Apache License 2.0
309 stars 119 forks source link

[BUG]: C++ Implementation of Deserialize is broken #1747

Closed mdemoret-nv closed 2 weeks ago

mdemoret-nv commented 2 weeks ago

Version

24.06

Which installation method(s) does this occur on?

Docker, Conda, Source, Kubernetes

Describe the bug.

In PR #1454, the C++ implementation of the deserialize node was disabled and has not been re-enabled. Changing the value of support_cpp_node to return True throws an exception due to incorrect C++ node names. Fixing this results in CUDA errors down the line.

Minimum reproducible example

Make the changes shown [here](https://github.com/nv-morpheus/Morpheus/pull/1739/files#diff-f3ac53e0a1506b4fa8c2d577afc95fa9d72c450e1e9efd9ad14a83b4eee8c7f1R116-R128) and run the ABP test: `pytest --run_slow ./tests/test_abp.py`

Relevant log output

Click here to see error details

 [Paste the error here, it will be hidden by default]

Full env printout

Click here to see environment details

 [Paste the results of print_env.sh here, it will be hidden by default]

Other/Misc.

No response

Code of Conduct