Skip to content

Conversation

@njacazio
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 10, 2025

O2 linter results: ❌ 10 errors, ⚠️ 9 warnings, 🔕 0 disabled

@github-actions github-actions bot changed the title Split eventfiltering cmake files [PWGCF,PWGEM,PWGHF,PWGJE,PWGLF,PWGUD,Trigger] Split eventfiltering cmake files Nov 10, 2025
@mpuccio
Copy link
Collaborator

mpuccio commented Nov 10, 2025

Hi @njacazio, what is the problem you are trying to solve here?

@njacazio
Copy link
Collaborator Author

Ciao @mpuccio no problem actually. Thanks to the recent feature of the selection of the installed target I wanted to split the compilation of core filtering from the user tasks. This is still a draft as you see. If you find it useful we can go on.

@nzardosh nzardosh self-requested a review November 10, 2025 12:01
@mpuccio
Copy link
Collaborator

mpuccio commented Nov 10, 2025

@njacazio, I see, for this I would see it useful only to decouple Zorro from the rest, but I would not split the various PWG as for maintaining the EventFiltering we want to have all the filters available without knowing which PWG we should manually enable. I can take care of the decoupling of Zorro, but I would not go further with the rest of the PWG decoupling.

@njacazio
Copy link
Collaborator Author

Hi, sounds good, please go ahead!

@njacazio njacazio closed this Nov 10, 2025
@njacazio njacazio deleted the nj-event-filtering branch November 10, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants