Draft: Add pixel reconstruction to Rich

assigned to @caschbur

mentioned in merge request LHCb!4629

added RTA label

added 1 commit

c724681a - Fixed formatting

Compare with previous version

changed the description

added 5 commits

c724681a...5efedce4 - 2 commits from branch 2024-patches
0e523492 - Added code to dump RichPDInfo
8cb0f9dd - Added condition derivation for PDInfo
5f288cc3 - Fixed formatting

Compare with previous version

added 1 commit

795f51ac - Modified the way the RichSmartIDs are printed, added methods to support this

Compare with previous version

Nice to see this @caschbur. If you have preliminary studies to show, feel free to present in an upcoming WP2 meeting.

Thank you, this is still a bit early but when it is more complete I think that would be a great idea.

What is the status here? How about scheduling a presentation until the end of the year?

Hi! We are currently working on getting the RichPD geometric information, so we're studying how to do a binary dumping of this.

After we are done with that, we will be able to start running and testing the code to see how it's looking, I hope we will have the dump sorted soon but after that it's hard for me to tell if the code will be pretty much ok or if it will need a bit of fixing, as we also have to make sure that the pixels created in Allen are the same as in HLT.

Maybe let me keep you informed so we can talk about the meeting?

Sounds good!

Note that for the geometry information, the important part is to have a consumer and producer for the geometry, as described here in the documentation.

In case it helps: It is not necessary to dump the information to a file, as you can also call Allen from Gaudi, so the geometry information will be obtained on the fly (second two options described here: https://allen-doc.docs.cern.ch/setup/run_allen.html).

Feel free to ask for help on the Allen developers mattermost channel for the geometry dumping and other questions.

Hi Dorothea, thanks for the links! We are progressing with the dumps and building the code, I hope we will have done this in about a week... And after that we get going with the running and testing.

I will be visiting CERN from November till the end of the year so I think I will be able to learn and progress faster over there :-).

Hi Josef, how are the developments going? Would you like to present the current status at a WP2 meeting in January? It would be interesting to take a checkpoint of this project ahead of the computing workshop.

Hello Dorothea! We've made a lot of progress, our dumpers turned out (seamingly) well and we also wrote tests for our sequence. However, we are fighting with a small error to actually be able to run our sequence so maybe give me one more week to see if everything ends up working well. If so, I will very happily present our progress! But I cant promise as of now that it will work perfectly before the end of the year (Though thats what I'm aiming for)

Great to hear about lots of progress having been made!

We will have no more WP2 meetings this year anyways due to the end-of-year holidays. How about scheduling a presentation on January 7th or 14th? You can see the planned WP2 meetings here: https://indico.cern.ch/category/10878/

Hello, happy new year! I'll have to apologize, we are still working on fixing the error that is preventing us from running the sequence. We have more detailed information on the origin of the bug, which seems to be in the process of loading and casting the geometry information. We are working hard on this and also asked for some help already so we will keep you updated as soon as we get it working! Thanks for your patience :P we are getting there

added 2 commits

b8832667 - initial RichPD.cuh file
f06a7f90 - Initial RichPDInfo.cuh file

Compare with previous version

changed the description

added 12 commits

f06a7f90...40a17aa8 - 4 commits from branch 2024-patches
aebbb0b4 - Added code to dump RichPDInfo
255b6143 - Added condition derivation for PDInfo
8c009ce9 - Fixed formatting
a9865574 - Modified the way the RichSmartIDs are printed, added methods to support this
ff6581a1 - initial RichPD.cuh file
672b4b3c - Initial RichPDInfo.cuh file
0784c3e9 - Started working on the make pixels algorithm
d13a9ab5 - Fixed conflicts

Compare with previous version

added 1 commit

dd877149 - Fixed formatting

Compare with previous version

added 17 commits

dd877149...e667708a - 6 commits from branch 2024-patches
56ebfa7e - 1 earlier commit
bbb8bb34 - Added condition derivation for PDInfo
bdbdb615 - Fixed formatting
d39ecbba - Modified the way the RichSmartIDs are printed, added methods to support this
2d506082 - initial RichPD.cuh file
352e7a0a - Initial RichPDInfo.cuh file
f18382a6 - Started working on the make pixels algorithm
f13d3c2f - Added code to dump RichPDInfo
82f88649 - Added condition derivation for PDInfo
66a3d51b - Fixed formatting
325d92ad - Fixed formatting

Compare with previous version

added 1 commit

1b7547e5 - Fixed CUDA compilation errors

Compare with previous version

added 14 commits

1b7547e5...81cc3017 - 2 commits from branch 2024-patches
81cc3017...1ff1fef9 - 2 earlier commits
71f10cb3 - Fixed formatting
4769aefe - Modified the way the RichSmartIDs are printed, added methods to support this
574385a2 - initial RichPD.cuh file
3f9bfbcc - Initial RichPDInfo.cuh file
d8fd4347 - Started working on the make pixels algorithm
63811233 - Added code to dump RichPDInfo
b8a360f5 - Added condition derivation for PDInfo
7d1e32f0 - Fixed formatting
9a6f7152 - Fixed formatting
c9b43e9d - Fixed CUDA compilation errors