Skip to content

Nj a3tofgeo#15125

Draft
njacazio wants to merge 5 commits intoAliceO2Group:devfrom
njacazio:nj-a3tofgeo
Draft

Nj a3tofgeo#15125
njacazio wants to merge 5 commits intoAliceO2Group:devfrom
njacazio:nj-a3tofgeo

Conversation

@njacazio
Copy link
Collaborator

@njacazio njacazio commented Mar 5, 2026

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5


#include "TRKReconstruction/ClustererACTS.h"
#include "TRKBase/GeometryTGeo.h"
#include <Acts/Clusterization/Clusterization.hpp>
Copy link
Contributor

@mpuccio mpuccio Mar 5, 2026

Choose a reason for hiding this comment

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

I think this is a very nice example of how to integrate ACTS in O2, I would still not add a clusterer that is matching line by line the other one we have now until we have one that uses the ACTS library

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ciao @mpuccio yes, this is the version I used to build against, indeed as we discussed before that's just a header call. I will of course have it different before moving the PR out of draft :) thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants