Skip to content

New detailed vtx digitizer#64

Open
jessy-daniel wants to merge 81 commits intokey4hep:masterfrom
jessy-daniel:New_Detailed_VTXdigitizer
Open

New detailed vtx digitizer#64
jessy-daniel wants to merge 81 commits intokey4hep:masterfrom
jessy-daniel:New_Detailed_VTXdigitizer

Conversation

@jessy-daniel
Copy link

@jessy-daniel jessy-daniel commented Jul 17, 2025

BEGINRELEASENOTES

  • Creation of a new class VTXdigiDetailed for a more detailled digitizer for vertex detector and tracker. it considers the deposit of charges inside sensors, their drift (due to magnetic and electric field and thermal diffusion), the collection of the charges in the sensor surface and the pixels clusterization. It returns a TrackerHitPlaneCollection to reconstruction.
  • This needs pixel segmentation in geometry which has been added only to CLD for now (see branch https://github.com/jessy-daniel/k4geo/tree/Add_pixels_tracker -> Will be under pull request too)

ENDRELEASENOTES

jessy-daniel and others added 30 commits January 15, 2025 17:18
…tracker (VTXDigiDetailed)

VTXDigiDetailed is currently a copy of VTXDigi + implementation of the primary_ionization function which creates the charges along the incoming particle path.
Note that for now VTXDigiDetailed runs exactly the same method than VTXDigi.
The goal will be to implement the full digitization of the hits, considering charge production and drift inside the active material and consider the readout response.
This needs a pixel segmentation of the corresponding layers in k4geo (see https://github.com/jessy-daniel/k4geo/tree/Add_pixels_tracker)
…es an issue in EDM4HEP->LCIO converter (to study)
@jessy-daniel
Copy link
Author

Hello @andresailer @BrieucF , I think that everything is fine now, I applied all the requested modifications. Sorry for the long time it took, I had a lot to do in other topics those past months...

@jmcarcell
Copy link
Member

pre-commit needs to be fixed, by running pre-commit and also clang-format: https://github.com/key4hep/k4RecTracker/actions/runs/21006810058/job/60393005406?pr=64

@jessy-daniel
Copy link
Author

pre-commit needs to be fixed, by running pre-commit and also clang-format: https://github.com/key4hep/k4RecTracker/actions/runs/21006810058/job/60393005406?pr=64

Hello, I did the formating

@jessy-daniel
Copy link
Author

The last commits were for debuging some last minutes bug I noticd when running the reconstruction steps after digitizer.
Everything should be ready now.

@jessy-daniel
Copy link
Author

Hi @jmcarcell @andresailer @BrieucF , could you plmease let me know if this can be merged now or what is missing ?
That would be great if it would be merged before my talk at FCC Physics workshop next week.
Best

@jessy-daniel
Copy link
Author

jessy-daniel commented Jan 20, 2026

Hi @jmcarcell, Thank you for all your help. I did the changes in the test files and CMakeList

@jmcarcell
Copy link
Member

Looks like the test doesn't pass

@jessy-daniel
Copy link
Author

Looks like the test doesn't pass

Hi @jmcarcell , yes. As the test works locally with ctest, I think this is due to the way we deal with paths in the makefile and test file, as git creates a job that probably mis-reproduce the paths. I will try to work on it later today

@jmcarcell
Copy link
Member

Try to run them in lxplus in a clean environment after sourcing the nightlies

@jessy-daniel
Copy link
Author

Try to run them in lxplus in a clean environment after sourcing the nightlies

It is exactly what I do and that works on lxplus. As said, I think this is just due to some paths that I will correct. Because the test is running, it just does not find the output file

@jessy-daniel
Copy link
Author

Dear @jmcarcell ; I hope this is OK now. Could you please try to launch the workflow ?
Best Regards

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants