Skip to content

Implement Core Library for building DeepROS nodes#15

Merged
thomasonzhou merged 22 commits intomainfrom
eddy/add_corelib
Sep 22, 2025
Merged

Implement Core Library for building DeepROS nodes#15
thomasonzhou merged 22 commits intomainfrom
eddy/add_corelib

Conversation

@Edwardius
Copy link
Contributor

@Edwardius Edwardius commented Sep 20, 2025

This PR implements the core Deep ROS library containing a base node handling backend plugin management and rclcpp_lifecycle.

Specifically it contains the following:
DeepNodeBase a base node that should be used by all other packages. It abstracts away the need for running inference with a backend to just a function run_inference.

Tensor a tensor data structure with abstracted memory management. It stores a pointer to the data which is managed and allocated by the specific backend. Basic Tensor operations are also available.

DeepBackendPlugin a abstract plugin interface used to define all future backend plugins. A DeepBackendPlugin must define how tensor data should be managed (using BackendMemoryAllocator abstract class) and how inference and model management should be ran (with BackendInferenceExecutor abstract class)

@Edwardius
Copy link
Contributor Author

FINALLY

Copy link
Contributor

@thomasonzhou thomasonzhou left a comment

Choose a reason for hiding this comment

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

Need some time to process.

Copy link
Contributor

@thomasonzhou thomasonzhou left a comment

Choose a reason for hiding this comment

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

Small comments

Copy link
Contributor

@thomasonzhou thomasonzhou left a comment

Choose a reason for hiding this comment

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

+1

@thomasonzhou thomasonzhou merged commit c920d36 into main Sep 22, 2025
2 checks passed
@thomasonzhou thomasonzhou deleted the eddy/add_corelib branch September 22, 2025 17:35
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