This repository has been archived by the owner on Jun 5, 2024. It is now read-only.
Set arg defaults #55
Merged
Set arg defaults #55
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks, @shenyangshi. Maybe you can also add this suggestion by Cecilia, for the distracted ones 😜: https://xenonnt.slack.com/archives/C0177G05M1P/p1653980781127549?thread_ts=1653633310.214119&cid=C0177G05M1P |
ramirezdiego
approved these changes
May 31, 2022
ramirezdiego
added a commit
that referenced
this pull request
Sep 29, 2022
* Harmonize with the latest WFSim/epix versions * Fix. * Fix. * Added a draft of the nT macro-clustering based on Micha's study * Fixed some calculation parameters, added a new example notebook * Add NR filter before quanta generation (#53) * New branch version. * NR-only cut string. * Fixed import of the NR cut config parameter * Fixed version label * nr cut with ER component smaller than 32kev * nr cut * field * change er threshold * add nr to bin run_epix * Add NR cut info to README * store_true for NR cut Co-authored-by: Pavel Kavrigin <[email protected]> Co-authored-by: Shenyang Shi <[email protected]> Co-authored-by: ramirezdiego <[email protected]> * Update HISTORY for release * Bump version: 0.2.2 → 0.3.0 * Set arg defaults (#55) * set nr_only cut default to false * add error for invalid input format * Save primary particle position (#56) * Added x_pri, y_pri, z_pri to the output * Debugged x_pri, y_pri, z_pri recording * WFSim requirements Co-authored-by: Diego Ramírez García <[email protected]> * Update HISTORY for release * Bump version: 0.3.0 → 0.3.1 * Fix the notebook + fix cS2 calculation. * First implementation of BBF quanta generator (#57) * Added BBF yield quanta generation in epix * Fix white space * updated readme * New intermediate results as defaults * Nex/Ni ratio for NR is computed inside and does not need to be provided * update nr parameters * add back fixed parameters in BBF fitting * Correct syntax Co-authored-by: Zihao Xu <[email protected]> Co-authored-by: Diego Ramírez García <[email protected]> * Update HISTORY for release * Bump version: 0.3.1 → 0.3.2 * fix conflict with master * removing import simualtor for merging with master * adding simualator to init Co-authored-by: Pavel Kavrigin <[email protected]> Co-authored-by: Shenyang Shi <[email protected]> Co-authored-by: ramirezdiego <[email protected]> Co-authored-by: Andrii Terliuk <[email protected]> Co-authored-by: Zihao Xu <[email protected]> Co-authored-by: Giovanni Volta <[email protected]>
ramirezdiego
added a commit
that referenced
this pull request
Mar 10, 2023
* Add NR filter before quanta generation (#53) * New branch version. * NR-only cut string. * Fixed import of the NR cut config parameter * Fixed version label * nr cut with ER component smaller than 32kev * nr cut * field * change er threshold * add nr to bin run_epix * Add NR cut info to README * store_true for NR cut Co-authored-by: Pavel Kavrigin <[email protected]> Co-authored-by: Shenyang Shi <[email protected]> Co-authored-by: ramirezdiego <[email protected]> * Update HISTORY for release * Bump version: 0.2.2 → 0.3.0 * Set arg defaults (#55) * set nr_only cut default to false * add error for invalid input format * Save primary particle position (#56) * Added x_pri, y_pri, z_pri to the output * Debugged x_pri, y_pri, z_pri recording * WFSim requirements Co-authored-by: Diego Ramírez García <[email protected]> * Update HISTORY for release * Bump version: 0.3.0 → 0.3.1 * First implementation of BBF quanta generator (#57) * Added BBF yield quanta generation in epix * Fix white space * updated readme * New intermediate results as defaults * Nex/Ni ratio for NR is computed inside and does not need to be provided * update nr parameters * add back fixed parameters in BBF fitting * Correct syntax Co-authored-by: Zihao Xu <[email protected]> Co-authored-by: Diego Ramírez García <[email protected]> * Update HISTORY for release * Bump version: 0.3.1 → 0.3.2 * fix fast sim * remove prints * including Jaron ss-ms res * commit berfore pull * decision tree clustering working * Specify not working numba in req. file (#62) * fix loop for * updated * add prim position for materials and e dep from G4 * fixing bugs * updates for the meeting * a very bad example * Update requirements.txt (#63) * Bump version: 0.3.2 → 0.3.3 * Sync hystory * Update python-publish.yml * Add zenodo badge * Set S2 clustering algorithm in the config * New example notebook --------- Co-authored-by: Pavel Kavrigin <[email protected]> Co-authored-by: Pavel Kavrigin <[email protected]> Co-authored-by: Shenyang Shi <[email protected]> Co-authored-by: ramirezdiego <[email protected]> Co-authored-by: Andrii Terliuk <[email protected]> Co-authored-by: Zihao Xu <[email protected]> Co-authored-by: Jaron Grigat <[email protected]> Co-authored-by: Joran R. Angevaare <[email protected]> Co-authored-by: Joran Angevaare <[email protected]> Co-authored-by: PavelKavrigin <[email protected]> Co-authored-by: Pavel Kavrigin <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This sets nr_only and cut_by_eventid to false by default, thus solves the issue raised by @cecilia-ferrari .