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

PR update with latest k4geo master branch & adding IDEA DRC simulation #3

Merged
merged 19 commits into from
Apr 23, 2024

Conversation

swkim95
Copy link
Owner

@swkim95 swkim95 commented Apr 23, 2024

PR update with latest k4geo master branch & adding IDEA DRC simulation

BrieucF and others added 19 commits March 21, 2024 11:55
…hi (key4hep#332)

* implement version of ALLEGRO with ECAL barrel with cell corners projective in phi

* Update FCCee/ALLEGRO/compact/ALLEGRO_o1_v03/ALLEGRO_o1_v03.xml

* add test for ALLEGRO o1 v03

* fix README

* improve the comments

* add detector construction v03

* updade also other xml files for inclined ecal barrel v03. Improve comments in xml of v02. In cpp of v03 add check that electrode length in xml is consistent with calo r1 r2 and inclination angle

* include units in cpp

---------

Co-authored-by: Brieuc Francois <[email protected]>
…RC detector constructors, plugins for running simluation with IDEA DRC using ddsim
@swkim95 swkim95 merged commit ad54c67 into DRC_geo_migrate_230918 Apr 23, 2024
2 of 11 checks passed
@swkim95 swkim95 deleted the PRupdate_240408 branch June 1, 2024 07:06
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.

5 participants