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.

Sign up for GitHub

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

Jump to bottom

Find obselete 20" Collection Efficiency fudge factor in tuning and remove it #6

Open
marc1uk opened this issue Oct 3, 2018 · 1 comment
Open

Find obselete 20" Collection Efficiency fudge factor in tuning and remove it #6

marc1uk opened this issue Oct 3, 2018 · 1 comment
Labels
Intermediate PMT

Comments

@marc1uk
Copy link
Owner

marc1uk commented Oct 3, 2018

see WCSim#231
@marc1uk
Copy link
Owner Author

marc1uk commented Oct 4, 2018

In old WCSim collection efficiency wasn't accounted for - effectively set to 100%.
When adding new B&L PMTs, Hamamatsu gave data both of overall relative efficiencies ( = QE x CE x HE) and values for specific QE, HE and CEs for the new (and old) PMTs.

B&L PMTs were added with all appropriate properties, but for old 20" PMTs CE had effectively been absorbed into 'tuning'.
To leave 20" results unchanged, but still give the correct relative response of B&L vs 20", B&L QEs had to be artificially increased.

=============

However.

  • ANNIE is not using either SK's old 20" PMTs, nor their B&L PMTs.
  • As long as all new ANNIE PMT definitions have appropriate relative QE, CE and HEs, we don't need any fudge factors in there.
  • We just need to make sure that the SK 20" CE, absorbed somewhere into 'tuning', gets removed or universally accounted for.

Open question: where is this tuning?

@marc1uk marc1uk changed the title Introduce inverse fudge factors to QE or remove fudge factor Find obselete 20" Collection Efficiency fudge factor in tuning and remove it Oct 4, 2018
marc1uk pushed a commit that referenced this issue Jun 20, 2022
@mnieslony
ETEL & LUX holders (#6)
5ae94ae 
* added v1 of ETEL+LUX holders
* fixed rotation of ETEL/LUX holders
* fixed typo in WCSimConstructANNIECylinderScan
* fixed typos in ConstructMaterials & ConstructANNIECylinderScan
* make linerreff consistent with other reflectivities
* slightly adjusted holder positions to remove geometry overlaps
* ANNIE p2v7 (#2)
* added new PMT type positionings
* fix inner structure rotation
rotate gdml inner structure in line with rotation of bottom cap PMTs introduced in the last commit.
* minor tidy ups; change colour of particles for white background, speed up drawing in visOGLSX.mac, make WCSimWCHit::SetMaxPE static (also used in drawing only).
* Added ANNIEp2v7 (PMT positions from scan), ANNIE holders, reflectivity tuning parameters
* removed some cout statements
* Added LUX/ETEL housings, adjusted black sheet dimensions
authored-by: Michael Nieslony <[email protected]>
marc1uk pushed a commit that referenced this issue Dec 7, 2022
@mnieslony
RATPAC comparison (relies on PR #6) (#9)
0d80d57 
* Added RATPAC comparison parameterizations

* updated setup script for cluster

* fixed missing line in WCSimPrimaryGeneratorAction.cc

* added missing header file in WCSimPrimaryGeneratorAction

* removed not needed WCSimPhysicsList.hh

* made some options about material properties configurable

* changed default values for ratpac tuning parameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Intermediate PMT
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@marc1uk