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

Fax update for campaign #501

Closed
wants to merge 14 commits into from
Closed

Fax update for campaign #501

wants to merge 14 commits into from

Conversation

mcfatelin
Copy link
Contributor

New update of fax specifically for the use of the analysis campaign. The main changes are:

  1. PMT gains and gain sigmas are now from PMT WG, which accommodate the condition in the first science run.
  2. Change the s2_area_mean_top_fraction to the updated value in MC. See Issue#491
  3. Corrected the PMT after pulse probability. Previous one was 100 times higher.
  4. The time resolution of ion-residual PMT after pulse is added. Values are from JulienW
  5. Also the uniformly time-distributed PMT after pulse was implemented. See summary note for more details of such type of PMT after pulses. Because it can happen very close to the primary hit, it is potentially a important responsible for peak finding/classification/clustering efficiency loss.

However this will not be the last update of fax during the analysis campaign. The foreseen update on fax would be the implementation of multi-PE PMT after pulse generation.

… two order of magnitudes overestimated. Also in this updated configuration, the uniformly time-distributed AP is included.
…irst science run) are used as default. Also temporarily the gain_sigma is 0.35*gain. Later gain_sigmas will be updated according to values given by PMT WG.
Copy link
Contributor

@JelleAalbers JelleAalbers left a comment

Choose a reason for hiding this comment

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

Thanks for implementing these so quickly Qing! I tested by simulating the standard dummy waveforms, which works without problems.

In the future we probably want to move some stuff out of XENON1T.ini, it is getting rather unwieldy. For now this is the simplest though.

@JelleAalbers
Copy link
Contributor

Merged in other pull request

@feigaodm feigaodm deleted the fax_update_for_campaign branch June 28, 2017 19:50
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.

2 participants