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

Remove EventTrackQA configurations, keeping default ones. #1573

Closed
wants to merge 1 commit into from

Conversation

mfaggin
Copy link
Contributor

@mfaggin mfaggin commented Apr 3, 2024

No description provided.

Copy link

github-actions bot commented Apr 3, 2024

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2022-pp-apass4
async-2023-pbpb-apass
async-2023-pp-apass1
async-data
async-mc
async-2022-pp-apass6

@mfaggin
Copy link
Contributor Author

mfaggin commented Apr 3, 2024

@benedikt-voelkel not sure how to label it, I understand that you want it asap regardless the reco pass, period and year right? Then shall I tag it with

+async-label async-data, async-mc

?

@benedikt-voelkel
Copy link
Contributor

benedikt-voelkel commented Apr 12, 2024

Dear @mfaggin
we recently changed the logic of the AnalysisQC quite a bit. Could you please have a look? #1590
The configs are split into the single once corresponding to their executable.
Let me know if there are specific requirements or changes. Just contact me e.g. via mattermost in that case.
Thanks a lot!

In addition, this can probably be closed now, right? Please feel free to do so.

@benedikt-voelkel
Copy link
Contributor

The configuration files that are removed here do anyway not exist anymore.
Closing.

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