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

No-spacing MC production setup #421

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

Conversation

sarahgaiser
Copy link
Contributor

@sarahgaiser sarahgaiser commented Jan 13, 2025

Sharing my setup for non-spaced MC production, still WIP.

@sarahgaiser sarahgaiser changed the title Moving A' generation to MG5 No-spacing MC production setup Jan 13, 2025
@sarahgaiser sarahgaiser marked this pull request as draft January 13, 2025 12:46
@tongtongcao
Copy link
Contributor

For moller_slic_job.py, what is difference between this job script and existing script slic_job.py?
For signal_pulser_overlay_to_recon_job.py, I suggest to keep this script, and build a new script for no-spacing, like signal_pulser_overlay_to_recon_job_noSpacing.py. Besides, for no-spacing, all contents related to spacing could be removed.
For signal_spaced_to_readout_job.py, it seems that you want no-filtered signal events (num_hodo_hits=0), and overlay with pulser data, and then run job up to readout while no reconstruction. I am not clear why you need such job script without reconstruction.
For single_pulser_recon_job.py, I guess that this script is only for reconstruction with input from readout. If so, input setup should be modified for this script, and all contents related to hpstr should be removed. Besides, if you need a script speficied for recon, then the script could be common for all reconstruction, so the script could be called as recon_job.py or mc_recon_job.py (to be distinguished with data reconstruction).
For recon_job.py, it seems to be reconstruction, format conversion and analysis for data. I suggested to change its name as data_recon_ana_job.py. Besides, the line with datacnv could be removed and job description should be modified.

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