Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade python util to use SurfaceId enum #180

Open
AndrewEdmonds11 opened this issue Jul 8, 2024 · 2 comments
Open

Upgrade python util to use SurfaceId enum #180

AndrewEdmonds11 opened this issue Jul 8, 2024 · 2 comments
Assignees
Labels
art / Offline This requires some touching of art / Offline enhancement New feature or request python This is python-related

Comments

@AndrewEdmonds11
Copy link
Collaborator

At the moment, we need to know the magic numbers to get specific surfaces. It would be better if we could use the SurfaceId enum directly in the same what that we can use the enum of ProcessCode (see https://github.com/Mu2e/TrkAna/blob/main/tutorial/pages/genealogy.md). This probably means that SurfaceId needs to be added to pywrap but we have the ProcessCode example we can follow

@AndrewEdmonds11 AndrewEdmonds11 added enhancement New feature or request python This is python-related art / Offline This requires some touching of art / Offline labels Jul 8, 2024
@brownd1978
Copy link
Contributor

brownd1978 commented Jul 8, 2024 via email

@kutschke
Copy link
Contributor

kutschke commented Jul 8, 2024

See Offline PR 1300 (Mu2e/Offline#1300)

@sophiemiddleton sophiemiddleton self-assigned this Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
art / Offline This requires some touching of art / Offline enhancement New feature or request python This is python-related
Projects
Status: Ready
Development

No branches or pull requests

4 participants