Skip to content

refactor: Rename the surface transform -> localToGlobalTransform#4958

Merged
paulgessinger merged 6 commits intoacts-project:mainfrom
junggjo9:transformToLocalToGlobal
Jan 15, 2026
Merged

refactor: Rename the surface transform -> localToGlobalTransform#4958
paulgessinger merged 6 commits intoacts-project:mainfrom
junggjo9:transformToLocalToGlobal

Conversation

@junggjo9
Copy link
Contributor

@junggjo9 junggjo9 commented Jan 13, 2026

  • Rename all occurances of the surface transform -> localToGlobalTransform
  • Introduce localToGlobalTransform method in the IDetectorElement
  • Mark the transform() methods as deprecated

--- END COMMIT MESSAGE ---

Towards #4467

  • GeoModelConverters should be refactored to accept the geometry context in a follow-up PR

Tagging: @paulgessinger, @asalzburger, @andiwand, @AJPfleger, @dimitra97

@github-actions github-actions bot added this to the next milestone Jan 13, 2026
@github-actions github-actions bot added Component - Core Affects the Core module Component - Fatras Affects the Fatras module Component - Examples Affects the Examples module Component - Plugins Affects one or more Plugins Vertexing Track Fitting labels Jan 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

📊: Physics performance monitoring for 9147983

Full contents

physmon summary

andiwand
andiwand previously approved these changes Jan 14, 2026
@benjaminhuth
Copy link
Member

Do we really want to overload Surface::localToGlobal with once returning a Vector, and the other time returning a Matrix? Sounds like an anti-pattern to me.

andiwand
andiwand previously approved these changes Jan 14, 2026
@junggjo9 junggjo9 force-pushed the transformToLocalToGlobal branch from c95a8c3 to 9147983 Compare January 15, 2026 08:28
@acts-policybot acts-policybot bot dismissed andiwand’s stale review January 15, 2026 08:28

Invalidated by push of 9147983

@sonarqubecloud
Copy link

@paulgessinger paulgessinger changed the title refactor: Rename the surface transform -> localToGlobal refactor: Rename the surface transform -> localToGlobalTransform Jan 15, 2026
@paulgessinger paulgessinger merged commit 0e749cc into acts-project:main Jan 15, 2026
42 of 44 checks passed
@junggjo9 junggjo9 deleted the transformToLocalToGlobal branch January 15, 2026 13:08
@acts-project-service
Copy link
Collaborator

acts-project-service commented Jan 15, 2026

🔴 Athena integration test results [0e749cc]

Build job with this PR failed!

Please investigate the build job for the pipeline!

@acts-project-service acts-project-service added the Breaks Athena build This PR breaks the Athena build label Jan 15, 2026
kodiakhq bot pushed a commit that referenced this pull request Jan 16, 2026
…e localToGlobalTransform in the child classes (#4970)

- localToGlobalTransform is the primary interface method since #4958
- Introduce a isSensitive() method to the DetectorElementBase & the Surface
@andiwand andiwand modified the milestones: next, v45.0.0 Jan 29, 2026
@andiwand andiwand mentioned this pull request Feb 2, 2026
2 tasks
JustusRudolph pushed a commit to JustusRudolph/acts that referenced this pull request Feb 23, 2026
…s-project#4958)

- Rename all occurances of the surface transform -> localToGlobalTransform
- Introduce localToGlobalTransform method in the IDetectorElement
- Mark the transform() methods as deprecated
JustusRudolph pushed a commit to JustusRudolph/acts that referenced this pull request Feb 23, 2026
…e localToGlobalTransform in the child classes (acts-project#4970)

- localToGlobalTransform is the primary interface method since acts-project#4958
- Introduce a isSensitive() method to the DetectorElementBase & the Surface
JustusRudolph pushed a commit to JustusRudolph/acts that referenced this pull request Feb 24, 2026
…s-project#4958)

- Rename all occurances of the surface transform -> localToGlobalTransform
- Introduce localToGlobalTransform method in the IDetectorElement
- Mark the transform() methods as deprecated
JustusRudolph pushed a commit to JustusRudolph/acts that referenced this pull request Feb 24, 2026
…e localToGlobalTransform in the child classes (acts-project#4970)

- localToGlobalTransform is the primary interface method since acts-project#4958
- Introduce a isSensitive() method to the DetectorElementBase & the Surface
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaks Athena build This PR breaks the Athena build Component - Core Affects the Core module Component - Examples Affects the Examples module Component - Fatras Affects the Fatras module Component - Plugins Affects one or more Plugins Track Fitting Vertexing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants