Skip to content

Conversation

@Li-Jiaoyang97
Copy link

This PR is saving two config files for the workflow: gen_g4_detsim_reco1 -> reco2_caf for the future production purpose.

@Li-Jiaoyang97 Li-Jiaoyang97 requested a review from henrylay97 July 13, 2023 21:24
@henrylay97
Copy link
Member

I fully approve of us committing some of this! I'm going to hold off on officially approving it until we have MCP2023B up and running over the coming few days. At that point @VCLanNguyen can add a couple of extra commits to this with all the new variants of the cfgs she has been producing for our new workflows 😄

@miquelnebot
Copy link

@henrylay97 @Li-Jiaoyang97 Updates on this? can be closed/merged?

Copy link
Member

@henrylay97 henrylay97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry Miquel. Yeah no reason not to merge this. I've initiated a slack discussion on the other things that I thought we should commit at the time of MCP2023B but that can form a separate PR if needed!

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.

4 participants