Skip to content

Naming Conventions - #427

Open
dellaert wants to merge 3 commits into
masterfrom
fix/naming
Open

dellaert wants to merge 3 commits into
masterfrom
fix/naming

Conversation

@dellaert

@dellaert dellaert commented Jun 15, 2026 •

Copy link
Copy Markdown
Member

In preparation for moving selected GTDynamics code into GTSAM, this PR updates names to comply with GTSAM conventions. The changes are intended to be mechanical and should not alter behavior. @yetongatech, please confirm you are comfortable with these conventions and let me know about any outstanding branches that could conflict.

dellaert added 2 commits June 15, 2026 10:17
…nd consistency

- Renamed `BasisKeyFunc` to `BasisKeyFunction` for better readability.
- Updated various instances of `params->retractor_creator` to `params->retractorCreator` for consistency across the codebase.
- Changed references from `TspaceBasis` to `TangentSpaceBasis` to reflect naming conventions.
- Modified test cases to align with new naming conventions and improved parameter structures.
- Enhanced readability by using camelCase for parameter names and method calls.
- Ensured that all tests related to the constraint manifold and its components are updated accordingly.
@dellaert
dellaert requested a review from yetongumich June 15, 2026 09:54
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