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

PMT gains in fax updated for SR1 #606

Merged
merged 1 commit into from
Sep 5, 2017
Merged

Conversation

mcfatelin
Copy link
Contributor

A quick PR for updating the PMT masks/gains used in fax. The gain values are from Shingo's note

@pdeperio
Copy link
Contributor

pdeperio commented Sep 5, 2017

How can we easily switch between different configs, e.g. SR0 vs SR1?

@mcfatelin
Copy link
Contributor Author

Maybe in the next PR where we update more about fax for SR1, we need to create an separate ini file for SR0 in case people want to use. But in principle it is better to have ini file connected to database.

@pdeperio
Copy link
Contributor

pdeperio commented Sep 5, 2017

Soon we will consider to re-analyze SR0, including re-calculation of all fax deliverables. So would be nice to have in this PR an input file with config option similarly to other maps that already have run dependence.

@feigaodm feigaodm merged commit ec2b2c5 into master Sep 5, 2017
@feigaodm feigaodm deleted the fax_config_switch_to_sr1 branch September 5, 2017 21:49
@pdeperio
Copy link
Contributor

pdeperio commented Sep 14, 2017

For future reference, I guess this list was taken at time test_t = 1502383532 (GMT: Thursday, August 10, 2017 4:45:32 PM).

We may wish to later check the effect of using gains from beginning of SR versus end of SR as a systematic.

@pdeperio
Copy link
Contributor

Two .inis for toggling implemented in XENON1T/processing#43

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