CPAlgorithm: clean up PtEtaSelection and ObjectCutflow

Remove the postfix option from the PtEtaSelectionBlock and ObjectCutFlowBlock config blocks. This option was only used to provide a unique name for the algorithms when calling the blocks several times - this can be done internally by looking up the selectionName instead.

Also remove the makePtEtaSelectionConfig and makeObjectCutFlowConfig methods: these are completely equivalent to setting up the blocks directly and were not used anywhere in the common code.

added analysis-interface-change label

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/AsgAnalysisAlgorithms

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/AsgAnalysisAlgorithms/python/AsgAnalysisConfig.py

Adding @krumnack ,@tadej as watchers

added Analysis CPAlgorithms analysis-review-required main review-pending-level-1 labels

CI Result SUCCESS (hash a95621e9)

	Athena	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

Looks good, approving - L1

added review-approved label and removed review-pending-level-1 label

added analysis-review-expert label and removed analysis-review-required label

Hi, since it touches the algorithm, ask expert for double check, @krumnack would you please take a look? Best regards

I guess this is fine. In principle we are slated to remove all the make*Config functions at the end of the year. Approving for AR.

Nils [Analysis RC]

added analysis-review-approved label and removed analysis-review-expert label

merged

mentioned in commit 7744510e