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

feat: merging workflows for iter-cSF #82

Closed
wants to merge 30 commits into from

Conversation

mondalspandan
Copy link
Contributor

@mondalspandan mondalspandan commented Nov 14, 2023

  • workflow : unified W+c & semilep electron/muon channel into one workflow
  • workflow : unified DY electron/muon channel into one workflow
  • util: array_writer to wrap common items

Ming-Yan and others added 21 commits October 12, 2023 20:07
- metadata:remove bad files
- scripts: implement remove bad files in fetch.py script
- plot: fix wrong axis defintion & add new variables definitions
- helpers: update xsections
- wf: add pv info & weight info in BTA_ttbar_wf
Conflicts:
	metadata/test_bta_run3.json
- ci: only trigger if changes
- doc,ci: add customize commit message to run ci
- wf: fix dtype bug
- ci: only trigger if changes
- doc,ci: add customize commit message to run ci
- wf: fix dtype bug
Changes to which variables are looked for, changes in output structure, etc.
@mondalspandan mondalspandan marked this pull request as ready for review November 14, 2023 15:09
@github-actions github-actions bot removed the CI label Nov 14, 2023
@uttiyasarkar uttiyasarkar self-requested a review November 16, 2023 12:57
@mondalspandan
Copy link
Contributor Author

Updated PR to also merge DY e and DY mu workflows

@Ming-Yan Ming-Yan changed the title Merging W+c, semilep ttbar workflows feat: merging workflows Nov 17, 2023
@Ming-Yan Ming-Yan changed the title feat: merging workflows feat: merging workflows for iter-cSF Nov 20, 2023
@uttiyasarkar
Copy link
Collaborator

Merging workflows moved to a new PR with additional changes.

Please find the new PR#85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants