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

Using LarG4Detector v6 in fdhd 1x2x2 #132

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

emanuele-villa
Copy link
Member

Depends on DUNE/dunesim#83 being approved. Using the new geometry v6 table instead of the v4 one.

@aolivier23
Copy link
Contributor

trigger build

aolivier23
aolivier23 previously approved these changes Jan 31, 2025
Copy link
Contributor

@aolivier23 aolivier23 left a comment

Choose a reason for hiding this comment

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

Small .fcl changes that shouldn't break the build. I will test the reconstruction with CI before releasing.

@FNALbuild
Copy link
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Collaborator

❌ CI build for DUNE Failed at phase ci_tests DUNE on slf7 for c14:prof - ignored failure for unit_test -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

@aolivier23 aolivier23 self-requested a review January 31, 2025 16:51
@aolivier23 aolivier23 dismissed their stale review January 31, 2025 16:52

CI broke when I checked it.

@aolivier23
Copy link
Contributor

Something is not quite right with the way these .fcl files affect other detector services. You can see an example failed test at https://dbweb8.fnal.gov:8443/LarCI/app/ns:dune/storage/docs/2025/01/31/test_DetectorProperties.log The full list of failed tests is at https://dbweb8.fnal.gov:8443/LarCI/app/ns:dune/build_detail/phase_details?build_id=dune_ci/18696&platform=Linux%20slf7&phase=unit_test&buildtype=slf7%20c14:prof

@FNALbuild
Copy link
Collaborator

❌ CI build for DUNE Failed at phase ci_tests DUNE on slf7 for e26:prof - ignored failure for unit_test -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

@emanuele-villa
Copy link
Member Author

emanuele-villa commented Feb 4, 2025

The fact that that table can't be found might be related to the fact that the dunesim version which dunecore is building against does not include dunesim MR # 83. A build against tagged version v10_03_01d02 should work. Do you agree?

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.

4 participants