Skip to content

Fields removal#133

Merged
lubynets merged 2 commits intoHeavyIonAnalysis:masterfrom
lubynets:fields_removal
Aug 5, 2024
Merged

Fields removal#133
lubynets merged 2 commits intoHeavyIonAnalysis:masterfrom
lubynets:fields_removal

Conversation

@lubynets
Copy link
Copy Markdown
Contributor

@lubynets lubynets commented Aug 5, 2024

  1. Enabled removing of user-defined fields from the config of the output branch (can be useful in AnalysisTree post-processing tasks such as Pid, Centrality, BranchWizard etc.).
    Note: this feature was not tested extensively, only basic checks were done. So you are welcome to test it, and before that use feature carefully :)
  2. Fixed bug introduced in PR #125 : variable 'matching_case' was added each time when EventHeader branch was attached to the output BranchConfig, that was not OK. Now it is fixed.

bugfix PR HeavyIonAnalysis#125: don't create matching_case var multiple times
@lubynets lubynets merged commit 82fc593 into HeavyIonAnalysis:master Aug 5, 2024
@lubynets lubynets deleted the fields_removal branch August 5, 2024 08:45
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.

1 participant