Skip to content

feat: Add TGeoAxes python bindings#5277

Merged
kodiakhq[bot] merged 4 commits intoacts-project:mainfrom
knutzk:kzoch-alignable-det
Mar 25, 2026
Merged

feat: Add TGeoAxes python bindings#5277
kodiakhq[bot] merged 4 commits intoacts-project:mainfrom
knutzk:kzoch-alignable-det

Conversation

@knutzk
Copy link
Copy Markdown
Contributor

@knutzk knutzk commented Mar 23, 2026

Adds python bindings for the TGeoAxes class that were previously missing. Introduces bindings for initialization via strings, value retrieval, string formatting (repr, str), and equality operators (eq, ne), along with implicit string conversions.

--- END COMMIT MESSAGE ---

The code was missing python bindings for the TGeoAxes class, which I noticed when introducing alignable-detector flags on the ODD full-chain example. The missing bindings (and a missing python include) are now added to the code, and using something like

getOpenDataDetector(odd_dir=geoDir, materialDecorator=oddMaterialDeco, misaligned=True)

is now possible.

@github-actions github-actions bot added this to the next milestone Mar 23, 2026
@knutzk knutzk marked this pull request as ready for review March 23, 2026 13:37
@paulgessinger
Copy link
Copy Markdown
Member

/ci-bridge-run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 23, 2026

📊: Physics performance monitoring for c148147

Full contents

physmon summary

@paulgessinger
Copy link
Copy Markdown
Member

So previously this was converted inside the convertToElements method, and now it's bound explicitly. Not a bad idea.

@sonarqubecloud
Copy link
Copy Markdown

@paulgessinger
Copy link
Copy Markdown
Member

/rerun

@kodiakhq kodiakhq bot merged commit 295d547 into acts-project:main Mar 25, 2026
43 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.

3 participants