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

feat: Geant4 detector plugin, part 2 #1759

Merged

Conversation

asalzburger
Copy link
Contributor

This PR adds the Json infrastructure to read and write the proto detector description.

It includes #1758 and is blocked until this one is merged.

@asalzburger asalzburger added Component - Plugins Affects one or more Plugins 🛑 blocked This item is blocked by another item labels Dec 16, 2022
@asalzburger asalzburger added this to the next milestone Dec 16, 2022
@codecov
Copy link

codecov bot commented Dec 16, 2022

Codecov Report

Merging #1759 (654fcac) into main (ae2811f) will not change coverage.
The diff coverage is n/a.

❗ Current head 654fcac differs from pull request most recent head b84bc26. Consider uploading reports for the commit b84bc26 to get more accurate results

@@           Coverage Diff           @@
##             main    #1759   +/-   ##
=======================================
  Coverage   49.66%   49.66%           
=======================================
  Files         403      403           
  Lines       22406    22406           
  Branches    10240    10240           
=======================================
  Hits        11127    11127           
  Misses       4135     4135           
  Partials     7144     7144           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Dec 16, 2022

📊 Physics performance monitoring for b84bc26

Full report
Seeding: seeded, truth estimated, orthogonal
CKF: seeded, truth smeared, truth estimated, orthogonal
IVF: seeded, truth smeared, truth estimated, orthogonal
Ambiguity resolution: seeded, orthogonal
Truth tracking
Truth tracking (GSF)

Vertexing

Vertexing vs. mu
IVF seeded

IVF truth_smeared

IVF truth_estimated

IVF orthogonal

Seeding

Seeding seeded

Seeding truth_estimated

Seeding orthogonal

CKF

CKF seeded

CKF truth_smeared

CKF truth_estimated

CKF orthogonal

Ambiguity resolution

seeded

Truth tracking (Kalman Filter)

Truth tracking

Truth tracking (GSF)

Truth tracking

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good - lets wait for part one to go in

Plugins/Json/src/ExtentJsonConverter.cpp Outdated Show resolved Hide resolved
Plugins/Json/src/ExtentJsonConverter.cpp Show resolved Hide resolved
@asalzburger asalzburger force-pushed the feat-geant4-add-json-infrastructure branch from a8d8b29 to eec0b5b Compare January 9, 2023 08:51
@asalzburger asalzburger removed the 🛑 blocked This item is blocked by another item label Jan 9, 2023
@asalzburger
Copy link
Contributor Author

This one is now quite a lot smaller ...

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@asalzburger asalzburger merged commit f4ec72a into acts-project:main Jan 9, 2023
asalzburger added a commit that referenced this pull request Jan 11, 2023
This PR concludes (for the moment) the Geant4 based detector example, it
adds at the `Detector/Geant4` to the Examples, adds the python bindings.

It is currently blocked as it depends on #1758 and #1759 - the first two
parts of this development.

With this PR, relatively simple GDML files can be translated one-to-one
into a `TrackingGeometry`, it serves as a validation test bench, and
will also provide future translation into `Experimental::Detector` in
order to cross-validate.

The following shows an output of a 'light' version of the ODD tracker:

![Screenshot 2022-12-16 at 17 04
33](https://user-images.githubusercontent.com/26623879/208139664-7202fb92-1702-4a3d-94fd-826b4f7a564d.png)

![Screenshot 2022-12-16 at 17 05
01](https://user-images.githubusercontent.com/26623879/208139683-2d2110e9-29c1-4308-87e0-fbbfa70d4cfe.png)

This can be reproduced with the following sources:


[odd-light.zip](https://github.com/acts-project/acts/files/10247296/odd-light.zip)

This contains:
```sh
-rwxr-xr-x@ 1 salzburg  staff      991 Dec 16 16:58 gdml_geometry.py
-rwxr-xr-x@ 1 salzburg  staff     2156 Dec 16 16:57 gdml_material_recording.py
-rwxr-xr-x@ 1 salzburg  staff     1926 Dec 16 16:57 gdml_material_validation.py
-rw-r--r--@ 1 salzburg  staff  6434606 Dec 16 16:56 odd-light.gdml
-rw-r--r--  1 salzburg  staff    79113 Dec 16 16:55 odd-proto-detector.json
```

The `odd-light.gdml` is the GDML description of the detector, the
`odd-proto-detector.json` the structure and then are three run files.
@paulgessinger paulgessinger modified the milestones: next, v23.0.0 Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Plugins Affects one or more Plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants