EIC issueshttps://eicweb.phy.anl.gov/groups/EIC/-/issues2022-07-21T15:44:47Zhttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/13Certificate issue when downloading field map: CAfile:/etc/pki/tls/certs/ca-bu...2022-07-21T15:44:47ZWouter DeconinckCertificate issue when downloading field map: CAfile:/etc/pki/tls/certs/ca-bundle.crt```
Compact INFO ++ Converted subdetector:SolenoidCoilBarrel of type athena_SolenoidCoil
Compact INFO ++ Converted subdetector:SolenoidCoilEndcapP of type ref_SolenoidEndcap
Compact INFO ++ Converted subdete...```
Compact INFO ++ Converted subdetector:SolenoidCoilBarrel of type athena_SolenoidCoil
Compact INFO ++ Converted subdetector:SolenoidCoilEndcapP of type ref_SolenoidEndcap
Compact INFO ++ Converted subdetector:SolenoidCoilEndcapN of type ref_SolenoidEndcap
FileLoader INFO Cache
FileLoader INFO Downloading fieldmaps/EIC_v.2.0.3_Magnetic_Field_Map_2021_09_28_radial_coords_cm_T.Bmap.txt as hash 6567da5a4887ad9b with curl --retry 5 -f https://eicweb.phy.anl.gov/EIC/detectors/athena/uploads/fd64db33e1c99547ab1840d8847fd1c1/EIC_v.2.0.3_Magnetic_Field_Map_2021_09_28_radial_coords_cm_T.Bmap.txt -o fieldmaps/6567da5a4887ad9b
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- 0 0 0 0 0 0 0 0 --:--:-- --:--:-- 0 0 0 0 0 0 0 0 --:--:-- --:--:----:--:-- 0
curl: (77) error setting certificate verify locations: CAfile:/etc/pki/tls/certs/ca-bundle.crt CApath: none
FileLoader ERROR unable to run cmd curl --retry 5 -f https://eicweb.phy.anl.gov/EIC/detectors/athena/uploads/fd64db33e1c99547ab1840d8847fd1c1/EIC_v.2.0.3_Magnetic_Field_Map_2021_09_28_radial_coords_cm_T.Bmap.txt -o fieldmaps/6567da5a4887ad9b
FileLoader ERROR check command and retry
nightly> akshaya@narval1:~/projects/def-wdconinc/akshaya/EIC$
```https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/55Far Backward Arb8 error2022-06-30T22:06:55ZWouter DeconinckFar Backward Arb8 errorThis seems like a degenerate arb8.
```
#25 136.2 Error in <TGeoArb8::ComputeTwist>: Shape TGeoArb8 type Arb8: Vertices must be defined clockwise in XY planes. Re-ordering...
#25 136.2 Error in <TGeoArb8::ComputeTwist>: Shape TGeoArb8 typ...This seems like a degenerate arb8.
```
#25 136.2 Error in <TGeoArb8::ComputeTwist>: Shape TGeoArb8 type Arb8: Vertices must be defined clockwise in XY planes. Re-ordering...
#25 136.2 Error in <TGeoArb8::ComputeTwist>: Shape TGeoArb8 type Arb8: Malformed polygon with crossing opposite segments
#25 144.5 *** Shape TGeoArb8: TGeoArb8 ***
#25 144.5 point #0 : x= -66.27951 y=-2399.27697 z= -20.00000
#25 144.5 point #1 : x= -9.80000 y=-1486.50000 z= -20.00000
#25 144.5 point #2 : x= -9.80000 y=-1486.50000 z= -20.00000
#25 144.5 point #3 : x= -26.28605 y=-2400.00000 z= -20.00000
#25 144.5 point #4 : x= -66.27951 y=-2399.27697 z= 20.00000
#25 144.5 point #5 : x= -9.80000 y=-1486.50000 z= 20.00000
#25 144.5 point #6 : x= -9.80000 y=-1486.50000 z= 20.00000
#25 144.5 point #7 : x= -26.28605 y=-2400.00000 z= 20.00000
#25 144.5 Bounding box:
#25 144.5 *** Shape TGeoArb8: TGeoBBox ***
#25 144.5 dX = 0.00000
#25 144.5 dY = 0.00000
#25 144.5 dZ = 0.00000
#25 144.5 origin: x= 0.00000 y= 0.00000 z= 0.00000
```Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/12dRICH sensor material should not be AirOptical2022-07-27T04:12:53ZChristopher DilksdRICH sensor material should not be AirOpticalThis was temporary and needs to be updated
- moreover, I had to add reflectivity properties to the dRICH sensor surface from the ATHENA `master` branch, in order to see hits; not sure if this is correct (since we did not have this in the...This was temporary and needs to be updated
- moreover, I had to add reflectivity properties to the dRICH sensor surface from the ATHENA `master` branch, in order to see hits; not sure if this is correct (since we did not have this in the ATHENA proposal production)
## tasks
- [ ] check the photon count, to make sure we get the expected numbers
- [ ] check the sensor geometry and segmentation
- [ ] surface paramaterization (esp. reflectivity)
- [ ] sensor bases (how to model the back of the sensors?) - make sure there is enough space for this
## notes from ATHENA studies (from discussions with Alexander, Chandra, and Chris)
- sensor used for proposal: https://www.hamamatsu.com/us/en/product/optical-sensors/mppc/mppc_mppc-array/S13361-3050AE-08.html
- the non-negligible gas-to-resin (numerically
<5%) and resin-to-silicon (~15%) reflections must be effectively
accounted in the quoted PDEs. At all incident angles of interest
for us (up to 30 degree or so) there numbers get shared between
the polarization states, but average stays more or less the same.
- However in our case (g4dRIChOptics.hh) the sensor surface was defined as dielectric_metal with
some bogus imaginary refractive index parameters. This killed the
photons, and also I verified that the MC .root files contain only
the ones which were detected somewhere. So I temporarily changed the material to AirOptical, and added
a benign surface to the optical_metarials.xml database, and we are
back at ~10 npe. And "Chandra's number" for 350..650nm integral is
now ~72 or so.
- I think the correct way to account Cherenkov photon polarization is
to create a resin volume, and perhaps even a silicon volume inside it,
but renormalize the PDE, accounting for the normal incident losses.
- look into `G4SiPM` packageChandradoy Chatterjeechandradoy.chatterjee@ts.infn.itChandradoy Chatterjeechandradoy.chatterjee@ts.infn.ithttps://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/issues/46Correspondence between ReconstructedParticle and MCParticles2022-07-31T00:10:47ZZhoudunming TuCorrespondence between ReconstructedParticle and MCParticlesWe would like to have a way to access a specific reconstructed particle that has matched to the MCparticle, e.g., the scattered electron, and so on. This would allow studies for smearing/resolution effect, efficiency and fake rate for tr...We would like to have a way to access a specific reconstructed particle that has matched to the MCparticle, e.g., the scattered electron, and so on. This would allow studies for smearing/resolution effect, efficiency and fake rate for tracking, user-defined quantities at reconstructed level, etc.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/54Job Failed #662974: increase timeout in generate_prim2022-05-26T05:06:56ZWouter DeconinckJob Failed #662974: increase timeout in generate_primJob [#662974](https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/jobs/662974) failed for 8cb6388d3805dee0418c0c4f67392325ba3fe738:Job [#662974](https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/jobs/662974) failed for 8cb6388d3805dee0418c0c4f67392325ba3fe738:Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/athena/-/issues/152Fix issues introduced by ip6 changes2022-05-13T19:17:51ZWouter DeconinckFix issues introduced by ip6 changesWouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/10Re-scale the dRICH2022-05-30T17:30:37ZChristopher DilksRe-scale the dRICHRe-scale the dRICH envelope. For the first attempt, we can use the ECCE dRICH dimensions.Re-scale the dRICH envelope. For the first attempt, we can use the ECCE dRICH dimensions.Christopher DilksChristopher Dilkshttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/8Use correct BaBar fieldmap2022-06-10T23:04:39ZWouter DeconinckUse correct BaBar fieldmaphttps://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/issues/45Explicitly load CaloHitContribution whenever SimCalorimeterHit is used2022-04-25T01:15:00ZWouter DeconinckExplicitly load CaloHitContribution whenever SimCalorimeterHit is usedSee https://github.com/key4hep/k4FWCore/issues/82.
Whenever
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
]
```
we need to modify to
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
"EcalEndcapNHit...See https://github.com/key4hep/k4FWCore/issues/82.
Whenever
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
]
```
we need to modify to
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
"EcalEndcapNHitsContributions",
]
```Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/issues/84Explicitly load CaloHitContribution whenever SimCalorimeterHit is used2022-04-25T01:19:04ZWouter DeconinckExplicitly load CaloHitContribution whenever SimCalorimeterHit is usedSee https://github.com/key4hep/k4FWCore/issues/82.
Whenever
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
]
```
we need to modify to
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
"EcalEndcapNHit...See https://github.com/key4hep/k4FWCore/issues/82.
Whenever
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
]
```
we need to modify to
```python
sim_coll = [
"MCParticles",
"EcalEndcapNHits",
"EcalEndcapNHitsContributions",
]
```Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/91Memory leak in PodioDataSvc2022-04-30T15:18:55ZWouter DeconinckMemory leak in PodioDataSvcWouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/90Unsigned eicd::RawCalorimeterHit amplitude causes threshold issues2022-06-21T22:43:35ZWouter DeconinckUnsigned eicd::RawCalorimeterHit amplitude causes threshold issues`(rh.getAmplitude() - m_pedMeanADC) < thresholdADC` for negative `rh.getAmplitude() - m_pedMeanADC` will results in wraparound since `unsigned int - int` is an `unsigned int`.`(rh.getAmplitude() - m_pedMeanADC) < thresholdADC` for negative `rh.getAmplitude() - m_pedMeanADC` will results in wraparound since `unsigned int - int` is an `unsigned int`.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/1Reenable views when phy-scratch is back2022-04-11T21:05:08ZWouter DeconinckReenable views when phy-scratch is backDisabled because no phy-scratch tagged runners.
- [x] revert: 4fab09f600407a8726b2d988d94c4cab0a03b70a
- [x] revert: e04e092cbbf3a4ad28a1d87203f4a5ec022202beDisabled because no phy-scratch tagged runners.
- [x] revert: 4fab09f600407a8726b2d988d94c4cab0a03b70a
- [x] revert: e04e092cbbf3a4ad28a1d87203f4a5ec022202beWouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/89Fill association simID with `part.id()` instead of `part.getObjectID().index`2022-03-27T23:57:44ZWouter DeconinckFill association simID with `part.id()` instead of `part.getObjectID().index`Seems limited to a few instances only, but we are probably not writing associations everywhere we should be writing them.Seems limited to a few instances only, but we are probably not writing associations everywhere we should be writing them.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/compton/-/issues/1Implementing full EIC magnet set2022-04-03T20:13:19ZWouter DeconinckImplementing full EIC magnet setWe should expand the number of magnets that are currently implemented in the geometry from the single one that is there now. To get started, click on the "create merge request" link on this issue, then start a Web IDE (or check out a bra...We should expand the number of magnets that are currently implemented in the geometry from the single one that is there now. To get started, click on the "create merge request" link on this issue, then start a Web IDE (or check out a branch).Omar HassanOmar Hassanhttps://eicweb.phy.anl.gov/EIC/detectors/athena/-/issues/149Dump geometry CI job should also produce FF geometry separately2022-03-19T16:45:32ZWouter DeconinckDump geometry CI job should also produce FF geometry separatelyCurrently we produce root files with geometry output for all types of component-combinations, but nothing specific to FF. @ajentschCurrently we produce root files with geometry output for all types of component-combinations, but nothing specific to FF. @ajentschWouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/issues/42Job Failed #597011: fast_ff association failure2022-03-01T21:39:07ZWouter DeconinckJob Failed #597011: fast_ff association failureJob [#597011](https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/597011) failed for 8595e1ccbfee59b6df7c85120cb4359e50d7536b:Job [#597011](https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/597011) failed for 8595e1ccbfee59b6df7c85120cb4359e50d7536b:Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/86Raw (digitized) hit model redundant, use single unified RawHit2022-02-28T23:28:50ZSylvester JoostenRaw (digitized) hit model redundant, use single unified RawHitCurrently all digitized hit structures store the same basic information: 64bit cell ID, 32bit time stamp and 32bit value. There is no good reason for them to be separate classes. We should unify them into a single RawHit structure as thi...Currently all digitized hit structures store the same basic information: 64bit cell ID, 32bit time stamp and 32bit value. There is no good reason for them to be separate classes. We should unify them into a single RawHit structure as this will provide us with more flexibility down the road. And when needed we can add data structures that actually store different information as we go. Proposed structure:
```
RawHit:
Description: Raw detector hit storing one 32-bit word of information
Members:
- uint64_t cellID
- uint32_t timeStamp
- uint32_t value
```https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/53Update ZDC Material and Cracks2022-02-25T21:52:09ZJihee KimUpdate ZDC Material and CracksUpdate standard scintillator material and cracksUpdate standard scintillator material and cracksJihee KimJihee Kimhttps://eicweb.phy.anl.gov/EIC/eicd/-/issues/25C++17 compatibility for vector_utils.h2022-02-22T22:55:32ZSylvester JoostenC++17 compatibility for vector_utils.hvector_utils.h right now requires C++20 (concepts). This clashes with ROOT versions compiled with C++17 only. We can resolve this by having a fall-back implementation (vector_utils_legacy.h) that is loaded when Concepts are not available...vector_utils.h right now requires C++20 (concepts). This clashes with ROOT versions compiled with C++17 only. We can resolve this by having a fall-back implementation (vector_utils_legacy.h) that is loaded when Concepts are not available that is as close as possible to the version with concepts in functionality.Sylvester JoostenSylvester Joosten