EIC issueshttps://eicweb.phy.anl.gov/groups/EIC/-/issues2022-05-24T17:49:05Zhttps://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/issues/86Ecal benchmark fails but job does not fail2022-05-24T17:49:05ZWouter DeconinckEcal benchmark fails but job does not failE.g https://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/jobs/676341
Issue is related to edm4hep transition: `EcalBarrelImagingClusters.ID.value` does not exist anymore.
@vgawasE.g https://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/jobs/676341
Issue is related to edm4hep transition: `EcalBarrelImagingClusters.ID.value` does not exist anymore.
@vgawasWouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/94Ensure MRs from forks do not fail due to missing docker secrets2022-06-01T18:23:57ZWouter DeconinckEnsure MRs from forks do not fail due to missing docker secretsThis likely is just a matter of skipping the docker push when no secrets are available.This likely is just a matter of skipping the docker push when no secrets are available.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/issues/87Re-enable layer-based energy profile in imaging_ecal benchmarks2022-06-05T21:46:37ZWouter DeconinckRe-enable layer-based energy profile in imaging_ecal benchmarks- [ ] 27fa81442d5cba3634e6c70f92dcf734872e48c2
- [ ] 7268d02a51f983c1dfecac0c7592b1d288d06f1d
This will require some thinking about how to get `utils.py` `get_layers_data` to return the same structures.- [ ] 27fa81442d5cba3634e6c70f92dcf734872e48c2
- [ ] 7268d02a51f983c1dfecac0c7592b1d288d06f1d
This will require some thinking about how to get `utils.py` `get_layers_data` to return the same structures.https://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/15How can we identify downstream (in sh or py files) which features are include...2022-06-07T02:44:33ZWouter DeconinckHow can we identify downstream (in sh or py files) which features are included in the detector?Use cases:
* we want to run a job only when the geometry has MRICH, but not when the geometry has PFRICH.
* we want a python options file to include the imaging layer algorithms when we have an imaging calorimeter.
We are using JUGGLER_...Use cases:
* we want to run a job only when the geometry has MRICH, but not when the geometry has PFRICH.
* we want a python options file to include the imaging layer algorithms when we have an imaging calorimeter.
We are using JUGGLER_DETECTOR and JUGGLER_DETECTOR_VERSION, but that doesn't scale beyond acadia/other and athena/ecce, not more fine-grained.https://eicweb.phy.anl.gov/EIC/juggler/-/issues/95JugFast::MatchClusters needs updating for EDM4hep migration2022-06-10T18:30:37ZSylvester JoostenJugFast::MatchClusters needs updating for EDM4hep migrationhttps://eicweb.phy.anl.gov/EIC/detectors/ecce/-/issues/16Revert e20517c042f107fa539e7a77100ebba0f3ea63a8 (source dir pollution)2022-06-15T00:05:21ZWouter DeconinckRevert e20517c042f107fa539e7a77100ebba0f3ea63a8 (source dir pollution)e20517c042f107fa539e7a77100ebba0f3ea63a8 was introduced because downstream uses DETECTOR_PATH pointing to the source directory.e20517c042f107fa539e7a77100ebba0f3ea63a8 was introduced because downstream uses DETECTOR_PATH pointing to the source directory.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/detector_benchmarks/-/issues/63Job Failed #707754: intermittent field map errors2022-06-14T21:00:48ZWouter DeconinckJob Failed #707754: intermittent field map errorsJob [#707754](https://eicweb.phy.anl.gov/EIC/benchmarks/detector_benchmarks/-/jobs/707754) failed for c2501e8ac89d393cf7da919715a14e5ad96752f6:Job [#707754](https://eicweb.phy.anl.gov/EIC/benchmarks/detector_benchmarks/-/jobs/707754) failed for c2501e8ac89d393cf7da919715a14e5ad96752f6:https://eicweb.phy.anl.gov/EIC/juggler/-/issues/96Implement SimTrackerHitsMerger to overlay/merge background events2022-06-18T03:28:20ZWouter DeconinckImplement SimTrackerHitsMerger to overlay/merge background eventsSimTrackerHitsCollector is an example class implementation.
Specs:
- SimTrackerHitsMerger can take a number of input collections, along with probabilities for each to be picked,
- With probability, it merges one of each of the input col...SimTrackerHitsCollector is an example class implementation.
Specs:
- SimTrackerHitsMerger can take a number of input collections, along with probabilities for each to be picked,
- With probability, it merges one of each of the input collections into the output collection.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/97ClusterRecoCoG introduces NaN in cluster energies2022-07-05T15:09:06ZWouter DeconinckClusterRecoCoG introduces NaN in cluster energiesFor `tw == 0` the following [lines](https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugReco/src/components/ClusterRecoCoG.cpp#L199) introduce a `NaN`:
```c++
if (tw == 0.) {
warning() << "zero total weights encountered, yo...For `tw == 0` the following [lines](https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugReco/src/components/ClusterRecoCoG.cpp#L199) introduce a `NaN`:
```c++
if (tw == 0.) {
warning() << "zero total weights encountered, you may want to adjust your weighting parameter." << endmsg;
}
cl.setPosition(v / tw);
```Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/56ZDC rotation and alignment2022-06-22T14:01:25ZAlex JentschZDC rotation and alignmentAlex JentschAlex Jentschhttps://eicweb.phy.anl.gov/EIC/juggler/-/issues/99TrackProjector: generalize by passing target surface as option2022-06-25T03:39:33ZWouter DeconinckTrackProjector: generalize by passing target surface as optionThe current [TrackProjector](https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugTrack/src/components/TrackProjector.cpp) has some hardcoded assumptions, i.e. https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugTrack/src/componen...The current [TrackProjector](https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugTrack/src/components/TrackProjector.cpp) has some hardcoded assumptions, i.e. https://eicweb.phy.anl.gov/EIC/juggler/-/blob/master/JugTrack/src/components/TrackProjector.cpp#L128-129
```c++
if (pathlength_at_reflayer<0 && volume==29 && layer==4 && trackstate.hasCalibrated()) pathlength_at_reflayer = pathlength; // 2nd outter barrel
if (pathlength_at_reflayer<0 && pathlength>1700) pathlength_at_reflayer = pathlength; // endcap GEM layer
```
We should change this to be more generic.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/57BackwardsTagWin should not use Arb82022-06-30T16:47:24ZWouter DeconinckBackwardsTagWin should not use Arb8Arb8 cannot handle degenerate vertices reliably. Probably better to use ExtrudedPolygon.
https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/merge_requests/101 addresses this in the short term to avoid errors, but it required adding a fudge ...Arb8 cannot handle degenerate vertices reliably. Probably better to use ExtrudedPolygon.
https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/merge_requests/101 addresses this in the short term to avoid errors, but it required adding a fudge distance...Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/issues/89Job Failed #746693: draw_clusters is still called even without imaging calori...2023-10-05T21:23:01ZWouter DeconinckJob Failed #746693: draw_clusters is still called even without imaging calorimeterJob [#746693](https://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/jobs/746693) failed for cbda7ae77925e0b4cbcd6c99b3471399702181ac:Job [#746693](https://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/jobs/746693) failed for cbda7ae77925e0b4cbcd6c99b3471399702181ac:Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/issues/49Uncaught failure: connection refused on mc cat2022-07-06T04:48:32ZWouter DeconinckUncaught failure: connection refused on mc catSee e.g. https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/755245
```
mc: <ERROR> Unable to read from `S3/eictest/ATHENA/EVGEN/DIS/NC/5x41/minQ2=1/pythia8NCDIS_5x41_minQ2=1_beamEffects_xAngle=-0.025_hiDiv_vtxfix_1.hepmc...See e.g. https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/jobs/755245
```
mc: <ERROR> Unable to read from `S3/eictest/ATHENA/EVGEN/DIS/NC/5x41/minQ2=1/pythia8NCDIS_5x41_minQ2=1_beamEffects_xAngle=-0.025_hiDiv_vtxfix_1.hepmc`. Get "https://dtn01.sdcc.bnl.gov:9000/eictest/?location=": dial tcp 198.125.208.25:9000: connect: connection refused.
WARNING: File does not end with END_EVENT_LISTING, appending
Moving generator output to input/dis/5on41/minQ2=1/gen-dis_5x41_minQ2=1_100.hepmc
Cleaning up
dis event generation complete
```Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/eicd/-/issues/28Add an (Open Source) license?2022-07-18T17:30:48ZDmitry KalinkinAdd an (Open Source) license?The project lacks COPYING/LICENSE file.The project lacks COPYING/LICENSE file.https://eicweb.phy.anl.gov/EIC/NPDet/-/issues/87Add an (Open Source) license?2022-07-18T17:30:49ZDmitry KalinkinAdd an (Open Source) license?The project lacks COPYING/LICENSE file.
Similar to https://eicweb.phy.anl.gov/EIC/eicd/-/issues/28The project lacks COPYING/LICENSE file.
Similar to https://eicweb.phy.anl.gov/EIC/eicd/-/issues/28https://eicweb.phy.anl.gov/EIC/detectors/ip6/-/issues/58Add an (Open Source) license?2022-07-18T17:30:44ZDmitry KalinkinAdd an (Open Source) license?The project lacks COPYING/LICENSE file.
Similar to https://eicweb.phy.anl.gov/EIC/eicd/-/issues/28 and https://eicweb.phy.anl.gov/EIC/NPDet/-/issues/87The project lacks COPYING/LICENSE file.
Similar to https://eicweb.phy.anl.gov/EIC/eicd/-/issues/28 and https://eicweb.phy.anl.gov/EIC/NPDet/-/issues/87https://eicweb.phy.anl.gov/EIC/benchmarks/physics_benchmarks/-/issues/50Re-enable failures on failed JB x and Q2 resolution fits2022-10-03T16:15:36ZWouter DeconinckRe-enable failures on failed JB x and Q2 resolution fitsWhile cluster/track matching was not working, JB was not picking up nearly enough hadronic final state (assumption) and giving wildly incorrect x and Q2 values. Once we have cluster/track matching working again, and once JB is operating ...While cluster/track matching was not working, JB was not picking up nearly enough hadronic final state (assumption) and giving wildly incorrect x and Q2 values. Once we have cluster/track matching working again, and once JB is operating stably again, we should re-enable pipeline failures on fit failures.Wouter DeconinckWouter Deconinckhttps://eicweb.phy.anl.gov/EIC/NPDet/-/issues/88Implement reading from gzipped hepmc input2022-08-30T00:28:24ZWouter DeconinckImplement reading from gzipped hepmc inputhttps://eicweb.phy.anl.gov/EIC/detectors/athena/-/issues/153Material map for EPIC2023-07-09T03:46:31ZShujie LiMaterial map for EPICFirst version of material map for the tracking volume of EPIC detector in LBL design. No AC-LGAD yet.
Material surfaces from Geantino scan: ![geant_mat_map](/uploads/67928357584ab8e9b92b37801f76735a/geant_mat_map.png)
Geantino vs. materi...First version of material map for the tracking volume of EPIC detector in LBL design. No AC-LGAD yet.
Material surfaces from Geantino scan: ![geant_mat_map](/uploads/67928357584ab8e9b92b37801f76735a/geant_mat_map.png)
Geantino vs. material map comparison:
![Ratio_Val_geant_mat_X0_](/uploads/37888d68f72c70d089928c82060d2e20/Ratio_Val_geant_mat_X0_.png)
The outer supporting cylinder is projected to the last disk which is not ideal. Need to improve later.
PR on github/epic: https://github.com/eic/epic/pull/96
Material map file: [material-maps.cbor](/uploads/56b64d544442a93904488e7292aa509d/material-maps.cbor)Shujie LiShujie Li