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

Build tms with updated geometry for PDR #33

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

AsaNehm
Copy link

@AsaNehm AsaNehm commented Oct 31, 2024

With the PDR for TMS coming up, the design of TMS has been adapted. The number of thin and thick steel layers has been changed, with a third option (double layers) being added as well. In addition the horizontal modules will be studied and need to be implemented as well.
Also the number of number of scintillator bars per module changed from 48 to 32 with 6 instead of 4 modules now

@AsaNehm AsaNehm self-assigned this Oct 31, 2024
@AsaNehm
Copy link
Author

AsaNehm commented Nov 6, 2024

Allows for the generation of two different types of geometries: pure stereo and full hybrid (every third layer an horizontal layer). To switch between them change in line 409 the variable to True for hybrid and False for stereo

@jdkio
Copy link

jdkio commented Nov 6, 2024

Running small tests of 10 files of 1e15 pot each using ProcessND.py
https://landscape.fnal.gov/lens/view/job/[email protected] - stereo
https://landscape.fnal.gov/lens/view/job/[email protected] - hybrid
If this works, I'd like to go up to 100-1000 files to be really sure, and also plot interaction positions as a way of seeing the geometry.

Ideally, we'd make some pictures of the geometry to verify everything is adjusted the way we want. We've had geometry issues before, like where it was rotated 90 degrees. I've asked about plotting geometry efficiently on slack

It also sounds like we need to verify whether the reconstruction parameters may need adjustments. These test files could be used to validate that

@jdkio
Copy link

jdkio commented Nov 6, 2024

Here are some jobs that worked. Only ran genie and g4 stages. Any failures were due to issues copying flux files
[email protected] - stereo
[email protected] - hybrid

Running tmsreco over 50 files from the output above. This version has a fix to the bar sanity check and I verfied it runs over at least one stereo output file above. But it seems to fail on most hybrid files, see below. It's tagged kleykamp_2024-11-06_candidate and branch kleykamp_2024-11-06_tar_candidate
[email protected] - stereo
[email protected] - hybrid

The hybrid file errors are very sparse. It just says,

Sliced event 111 into 2 slices
not in TMS?? hit 77(backwards) : 218.005, -1473.48, 11315
not in TMS?? hit 78(backwards) : 218.005, -1490.74, 11250
terminate called without an active exception
Aborted (core dumped)

Note that "not in TMS??" messages may or may not be related. There are additional "not in TMS??" messages that don't crash. The message comes from TMS_Kalman. If there's no simple fix, we should make an issue to track this in dune-tms

The program doesn't crash at the same location if I turn off the kalman filter using the config file. But it still crashes with a seg fault:
ConvertToTMSTree.exe /pnfs/dune/scratch/users/kleykamp/nd_production/2024-11-06_test_geom_v104_hybrid_small_edep_only/edep/FHC/00m/00/neutrino.29_1730927987.edep.root

#6  0x00007f9f8d333eeb in TMS_TrackFinder::TrackMatching3D (this=this
entry=0x20a3b60 <TMS_TrackFinder::GetFinder()::Instance>) at /cvmfs/larsoft.opensciencegrid.org/products/gcc/v9_3_0/Linux64bit+3.10-2.17/i
nclude/c++/9.3.0/bits/std_abs.h:72
#7  0x00007f9f8d3405e3 in TMS_TrackFinder::FindTracks (this=this
entry=0x20a3b60 <TMS_TrackFinder::GetFinder()::Instance>, event=...) at TMS_Reco.cpp:523
#8  0x000000000040b326 in ConvertToTMSTree (filename=..., output_filename=...) at ../src/TMS_Reco.h:181
#9  0x0000000000408a86 in main (argc=<optimized out>, argv=<optimized out>) at ConvertToTMSTree.cpp:268

@ast0815
Copy link
Member

ast0815 commented Nov 7, 2024

Ideally, we'd make some pictures of the geometry to verify everything is adjusted the way we want. We've had geometry issues before, like where it was rotated 90 degrees. I've asked about plotting geometry efficiently on slack

I tried and failed to implement that. The next best solution I could come up with was the 3D browser thing at https://dune.github.io/dunendggd/

It will always have the latest build geometry, so you can inspect the PR build there. Though it only builds the "default geometries" at the moment. So when you did not change those, you have to load your own geometry files from your PC into it. Which is no problem at all, just click on the three dots next to the file selector.

@jdkio
Copy link

jdkio commented Nov 7, 2024

Thanks! I couldn't find the loading buttons. I had to adjust the center bar until the three dots became visible

@ast0815
Copy link
Member

ast0815 commented Nov 7, 2024

Let me know when this is actually ready for review. So far I assume that this is a WIP. Should maybe be marked as such.

@AsaNehm AsaNehm marked this pull request as draft November 13, 2024 15:33
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.

3 participants