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

Generalize process id producer and stitching configuration and include stitching for w_lnu samples #55

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pkausw
Copy link
Contributor

@pkausw pkausw commented Jan 10, 2025

This PR generalizes the modules relevant for stitching samples in the analysis. The main differences are two-fold:

  1. The process id producer is now more generalized to account for different methods of stitching. While this is probably overkill while we only stitch DY and W+jets samples, this might be useful in the future
  2. the config entry has a more generalized structure:
cfg.x.stitching = {
            "DATASET_TAG": {  # e.g. for DY datasets, this would be 'is_dy'
                "base_producer": PRODUCER_TO_USE,  # e.g. for DY, this would be `process_ids_dy`
                "info": {
                    "m50toinf": {
                        "inclusive_dataset": INCLUSIVE_DATASET,   # e.g. for DY, this would be `cfg.datasets.n.dy_m50toinf_amcatnlo`
                        "leaf_processes": [
                            # the following processes cover the full njet and pt phasespace
                            # for DY:
                            # procs.n.dy_m50toinf_0j,
                            # *(
                            #    procs.get(f"dy_m50toinf_{nj}j_pt{pt}")
                            #    for nj in [1, 2]
                            #    for pt in ["0to40", "40to100", "100to200", "200to400", "400to600", "600toinf"]
                            # ),
                            # procs.n.dy_m50toinf_ge3j,
                        ],
                    },
                }
            },
}

This way, we can configure all stitchings in one place. Tested with w_lnu and dy datasets

@pkausw pkausw added enhancement New feature or request priority-high High priority labels Jan 10, 2025
@pkausw pkausw self-assigned this Jan 10, 2025
riga added a commit that referenced this pull request Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant