SBNSoftware / sbndcode

11 stars 44 forks source link

As built CRT geometry, v1 #477

Closed djpayne-thing closed 3 months ago

djpayne-thing commented 3 months ago

Updates the CRT geometry to something a bit closer to what we built. Does not include gaps, etc, but this is more or less what exists. Does not include the skinny modules under the cryo valve. Top CRT is closer to what will actually exist than it was. It's a process.

bear-is-asleep commented 3 months ago

Hi @djpayne-thing , can you assign a reviewer please?

djpayne-thing commented 3 months ago

Looks like I can assign myself. Is that appropriate?

bear-is-asleep commented 3 months ago

It's typical to have someone else review, so there are multiple eyes on it. Are there any other CRT experts who can check the geometry? @henrylay97 comes to mind.

fjnicolas commented 3 months ago

Looks like I can assign myself. Is that appropriate?

@djpayne-thing it must be someone different than the PR author (actually GitHub doesn't allow you to do that). Probably simulation conveners or people previously working on geometry (e.g. @gustavogx @mrguzzo) are good options

djpayne-thing commented 3 months ago

gdml1 gdml2 This is what the changes I've made look llike

henrylay97 commented 3 months ago

This is really great, thanks @djpayne-thing!

Marco's comment is important for picking up the geometry updates when we run jobs.

@bear-is-asleep I will need to keep in the loop when this goes into a release so that I can make sure the channel mapping in that release matches this geometry.

Mainly for the record of the PR but also for my total clarity could you list what does change in this update? (I think it's adding the extra flat module, shifting the horizontal north wall modules on the west side down a little and adding the two square modules in the bottom corner? Is that right?)

djpayne-thing commented 3 months ago

Added a separate pull request for geometry_sbnd.fcl

djpayne-thing commented 3 months ago

I've realized there's an additional change relative to the original that should also be in here. I'll close this and make a fresh pull request