I'm thinking about the best approach for implementing pipelines based on existing workspaces (such as #27 and #28).
The primary goal is to ensure ease of use, while maintaining minimal code duplication.
For example, the VLP-16 to Husky pipeline could be simplified by using symbolic links and gitignore for vlp_ws and husky_ws, which will 100% reuse the code of those 2 workspaces without duplication. I don't think this is the best way though...
I’m currently thinking of introducing only the required code and configs for vlp_to_husky_ws, where:
vlp_to_husky_ws/src/dummy_controller/dummy_controller/publisher.py publishes the commands.
vlp_to_husky_ws/docker/compose.yaml extends the compose files from vlp_ws and husky_ws.
- and other necessary files.
This way, we can treat vlp_to_husky_ws just like a normal workspace (that depends on some workspaces) with minimal code duplication. Moreover, we can easily integrate existing workspaces such as gazebo_world_ws or isaac_sim_ws in the future.
I'm curious about @YuZhong-Chen's and @Assume-Zhan's thoughts on this. I look forward to your comments when you have time.
I'm thinking about the best approach for implementing pipelines based on existing workspaces (such as #27 and #28).
The primary goal is to ensure ease of use, while maintaining minimal code duplication.
For example, the VLP-16 to Husky pipeline could be simplified by using symbolic links and gitignore for
vlp_wsandhusky_ws, which will 100% reuse the code of those 2 workspaces without duplication. I don't think this is the best way though...I’m currently thinking of introducing only the required code and configs for
vlp_to_husky_ws, where:vlp_to_husky_ws/src/dummy_controller/dummy_controller/publisher.pypublishes the commands.vlp_to_husky_ws/docker/compose.yamlextends the compose files fromvlp_wsandhusky_ws.This way, we can treat
vlp_to_husky_wsjust like a normal workspace (that depends on some workspaces) with minimal code duplication. Moreover, we can easily integrate existing workspaces such asgazebo_world_wsorisaac_sim_wsin the future.I'm curious about @YuZhong-Chen's and @Assume-Zhan's thoughts on this. I look forward to your comments when you have time.