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

Capacitor pedestal calibration #239

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

georgievgeorgi
Copy link
Contributor

Linear regression sums are collected inside PadmeReco and are exported
to a json file. This behaveour can be enabled with GenADCCellCalib
keyword in PadmeReconstruction.cfg under [RUNNINGMODE] group. The
produced sums need to be interpreted by
utils/v1742-offline-cell-calib-linear-regression.py -- poly5 method is
slowest but provides best results. On a second iteration, the
calibration data can be provided to the reconstruction via
ADCCellCalibData keyword again under [RUNNINGMODE] group

Fixes #238

Linear regression sums are collected inside PadmeReco and are exported
to a json file. This behaveour can be enabled with `GenADCCellCalib`
keyword in PadmeReconstruction.cfg under `[RUNNINGMODE]` group. The
produced sums need to be interpreted by
utils/v1742-offline-cell-calib-linear-regression.py -- poly5 method is
slowest but provides best results.  On a second iteration, the
calibration data can be provided to the reconstruction via
`ADCCellCalibData` keyword again under `[RUNNINGMODE]` group

Fixes #238
@georgievgeorgi
Copy link
Contributor Author

@eleonardi the calibration is switched off by default. One can enable it by setting these variables in the config file
https://github.com/PADME-Experiment/padme-fw/blob/123b53328576b891431c49c24f54f7af73c656c9/PadmeReco/config.template/PadmeReconstruction.cfg#L27#L29

@georgievgeorgi georgievgeorgi marked this pull request as draft January 27, 2022 11:12
@georgievgeorgi
Copy link
Contributor Author

georgievgeorgi commented Jan 27, 2022

remove json dependence?

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.

Implement v1742 offline noise calibration
1 participant