Tags: eic/epic
Tags
add 10x130 interpolated beamline to enable normal usage like other en… ( #839) …ergies ### Briefly, what does this PR introduce? This PR introduces a 10x130 lattice which has been calculated via interpolation between the three "core" beam energy settings, while still keeping the B0 constant. This will be paired with a similar PR for the RP matrices in EICrecon. This updated magnet setting file will allow for "normal" usage of the file, where the B0 and RP can be used simultaneously. ### What kind of change does this PR introduce? - [ ] Bug fix (issue #__) - [ x] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ x] Tests for the changes have been added - [x] Documentation has been added / updated - [x ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? No, improves performance. ### Does this PR change default behavior? Yes, it changes the default 10x130 magnet settings to something which can be properly used.
Add Q3eR field rotation (#837) ### Briefly, what does this PR introduce? The geometry of the Q3eR magnet was rotated but the field volume wasn't rotated to match. ### What kind of change does this PR introduce? - [x] Bug fix (issue #__) - [ ] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ ] Tests for the changes have been added - [ ] Documentation has been added / updated - [ ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? No ### Does this PR change default behavior? No
Update material map (#827) ### Briefly, what does this PR introduce? update the link to new material map created by @bschmookler which addresses the artificial inconsistency introduced in 24.10 due to geometry change. See performance study at https://indico.bnl.gov/event/26553/ New map stored at https://eicweb.phy.anl.gov/EIC/detectors/athena/-/issues/153 as usual. ### What kind of change does this PR introduce? - [ x] Bug fix (issue #__) - [ ] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ x] Tests for the changes have been added - [ ] Documentation has been added / updated - [x ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? N/A ### Does this PR change default behavior? It will bring the tracking performance back to normal for any simulation with ePIC 24.10 or newer. Co-authored-by: Dmitry Kalinkin <[email protected]>
Added TaggerTrackerTransportation.onnx to calibrations.xml (#823) ### Briefly, what does this PR introduce? Adds the onnx implementation of the Low-Q2 reconstruction to the calibrations.xml for download. ### What kind of change does this PR introduce? - [ ] Bug fix (issue #__) - [x] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ ] Tests for the changes have been added - [ ] Documentation has been added / updated - [ ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? no ### Does this PR change default behavior? Downloads an additional onnx file.
backward_PbWO4: adjust geometry to match latest CAD design (#815) ### Briefly, what does this PR introduce? This updates tower geometry to match the latest CAD design. ### What kind of change does this PR introduce? - [ ] Bug fix (issue #__) - [ ] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ ] Tests for the changes have been added - [ ] Documentation has been added / updated - [ ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? No ### Does this PR change default behavior? Yes
fix(ci): don't trigger geoviewer directly (#794) ### Briefly, what does this PR introduce? Now that we trigger the geoviewer from the container job, along with other benchmarks, we don't need to trigger it directly (and it doesn't work to trigger it directly). E.g. https://eicweb.phy.anl.gov/EIC/benchmarks/geoviewer/-/jobs/3948345 triggered by https://github.com/eic/epic/actions/runs/11263980547/job/31323141281 just uses epic-main in the container, not the update-sensor-geometry-for-RP it should be using. This PR removes the direct trigger of the geoviewer. ### What kind of change does this PR introduce? - [x] Bug fix (issue: ) - [ ] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ ] Tests for the changes have been added - [ ] Documentation has been added / updated - [ ] Changes have been communicated to collaborators ### Does this PR introduce breaking changes? What changes might users need to make to their code? No. ### Does this PR change default behavior? No. Co-authored-by: Dmitry Kalinkin <[email protected]>
Update sensor geometry for rp (#791) ### Briefly, what does this PR introduce? This PR introduces changing RP geometry for the various beam energies, and updates the sensor size for the AC-LGADs to what we are planning to build. ### What kind of change does this PR introduce? - [ ] Bug fix (issue #__) - [ x] New feature (issue #__) - [ ] Documentation update - [ ] Other: __ ### Please check if this PR fulfills the following: - [ x] Tests for the changes have been added (see PDF) - [ x] Documentation has been added / updated - [x ] Changes have been communicated to collaborators Changes will be presented at meetings on Monday and Tuesday (Oct 14th and 15th) See slides here: https://www.dropbox.com/scl/fi/43mhwxeyvluku0w1ojqfr/new_roman_pots_geometry_10_9_2024.pdf?rlkey=w0wc0gzrk4d03e8uh9w8s0tq5&dl=0 ### Does this PR introduce breaking changes? What changes might users need to make to their code? No changes to user code needed, but users need to be careful to select the correct geometry when running dd4hep AND EICrecon. (https://chat.epic-eic.org/main/pl/rptqecge1jyctkkwa1454zy1mw) I will hopefully address this issue in a PR immediately following this one. ### Does this PR change default behavior? Yes, the low-pT acceptances at the Roman pots will be much closer to reality now, and will vary with beam energy, as they should. --------- Co-authored-by: Alexander Jentsch <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> (cherry picked from commit 21add61)
PreviousNext