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

Parallelized offline vertexing reconstruction #46663

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

cericeci
Copy link

@cericeci cericeci commented Nov 11, 2024

PR description:

This PR contains a first implementation of the parallelized offline vertexing using Alpaka. We intend this as a first PR to request feedback and comments. It includes contributions from @alexstrel that were squashed in during the porting to 14_2.

The content of the PR was previously discussed on:
https://indico.cern.ch/event/1442046/#4-status-of-offline-vertexing

PR validation:

When running the code-checks there are several warnings such as:

Processing tmp/el9_amd64_gcc12/code-checks/RecoVertex/PrimaryVertexProducer_Alpaka/plugins/alpaka/PrimaryVertexProducer_Alpaka.cc.yaml
Deleting: No Diagnostics found

which I'm not sure about their origin. Otherwise it properly compiles and (local) validation works. The added code should not touch any other packages.

@cmsbuild
Copy link
Contributor

cms-bot internal usage

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46663/42589

  • Found files with invalid states:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/testAlpaka.root:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/testCPU_noPU.py:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/testCPU_PU200.py:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/prevs/testCPU_PU0.root:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/prevs/testAlpaka.root:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/testPrimaryVertexProducer_Alpaka.py:
    • RecoVertex/PrimaryVertexProducer_Alpaka/test/testPrimaryVertexProducer_Alpaka_PU200.py:

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@cericeci
Copy link
Author

Sorry, seems like when I rebased this to 14_2 there was some mix up on my side with the validation branch. Is it ok to push on this branch or do I close and open a new PR from the proper one?

@makortel
Copy link
Contributor

You can push to the same branch.

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

Successfully merging this pull request may close these issues.

3 participants