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: add submission scripts and unified histogrammer #103

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

Conversation

Ming-Yan
Copy link
Collaborator

@Ming-Yan Ming-Yan commented Nov 5, 2024

This commits includes the change presented in POG meeting
https://indico.cern.ch/event/1450713/contributions/6156720/attachments/2937537/5160065/CommFW%20update.pdf
The changes are mostly wrapped in commit63a302 and propagated to all the workflows.

A few framework restructuring is also done:
61c0a3 and df3407f.
The later commit only includes file clean up in the metadata and data/.

@Ming-Yan
Copy link
Collaborator Author

Ming-Yan commented Nov 5, 2024

review can include partial commits, the restructure commit is heavy
image
The actions changes can be viewed in master branch https://github.com/Ming-Yan/BTVNanoCommissioning/tree/master, checks in PR will fail due to the changes in CI not propagated

@Ming-Yan Ming-Yan force-pushed the btv_master branch 2 times, most recently from 2812005 to 3001bda Compare November 11, 2024 16:28
@Ming-Yan Ming-Yan force-pushed the master branch 3 times, most recently from 3001bda to a85f574 Compare December 17, 2024 11:15
feat : update 2024
- create suball script, make runner more modulize
- update naming scheme
- update to all workflows

feat: unified system shifts
- utils: read wps from json files
- ci: add camapgin
- ci: add automation ci check
- scripts: add debug info
- wf: adapt campaign and years
- utils, helper: clean up JEC
- readme: update installation
- data: clean up  UL files
- data: change the directory name follow jsonpog integration
- metadata: nested structure of json files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants