Skip to content

refactor: rename graph_msf to holistic_fusion - #54

Merged
nubertj merged 1 commit into
mainfrom
julian/rename-graph_msf-to-holistic_fusion
Sep 28, 2026
Merged

nubertj merged 1 commit into
mainfrom
julian/rename-graph_msf-to-holistic_fusion

Conversation

@nubertj

@nubertj nubertj commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The library name matches the repo name holistic_fusion.
  • Mechanical rename over contents and paths: graph_msf → holistic_fusion, GraphMsf/GraphMSF → HolisticFusion, GRAPH_MSF → HOLISTIC_FUSION, "Graph MSF" → "Holistic Fusion".
  • Covers packages, directories, the C++ namespace, CMake targets, include paths, header guards, docs, CI and Docker scripts.

Breaking changes

  • Downstream code must update includes, the namespace, CMake targets and package names.
  • Default frame and topic names change too, e.g. world_graph_msf → world_holistic_fusion.
  • GraphMsfHolistic becomes HolisticFusionHolistic. A shorter name is a possible follow-up.

Test plan

  • git grep -iE 'graph[-_ ]?msf' finds nothing.
  • CI builds the ROS1 and ROS2 images and runs the regression tests.

🤖 Generated with Claude Code

Rename every graph_msf package, directory, namespace, CMake target,
include path, class, macro, frame and topic name. GraphMsf becomes
HolisticFusion, GRAPH_MSF becomes HOLISTIC_FUSION.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nubertj nubertj self-assigned this Sep 28, 2026
@nubertj
nubertj requested a review from tutunarsl September 28, 2026 17:50
@nubertj
nubertj marked this pull request as ready for review September 28, 2026 17:51

@nubertj nubertj left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

overall looks good, but a few changes don't seem to be a pure rename. Needs to be double checked

Comment thread holistic_fusion/include/holistic_fusion/interface/constants.h
Comment thread ros/graph_msf_ros/include/graph_msf_ros/constants.h
@nubertj
nubertj merged commit 9db3c26 into main Sep 28, 2026
4 checks passed
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.

1 participant