Edward Moyse authored 1 year ago and Tadej Novak committed 1 year ago

VP1: fix more warnings and UI issues

Add InDetServMat config, add default Geo TAG in output file name, disable temporarily not used CLI options

Riccardo Maria Bianchi authored 1 year ago and Frank Winklmeier committed 1 year ago

Fix the configuration of InDetServMat geometry in VP1

Make Filename optional
Fix AtlasExtrapolation error (ATLASVPONE-659)

Adapted to the GeoVolumeCursor interface change.

Also fixed compilation problem in VP1Utils/VP1JobConfigInfo

Beltran Fernandez Barbadillo authored 1 year ago and Tadej Novak committed 1 year ago

Add Trigger retrievers to TriggerRetrieversCfg (CA config) and merge CA in AlgoJiveXMLCfg

Christos Anastopoulos authored 1 year ago and Tadej Novak committed 1 year ago

cppcheck/clang 'perfomance' fixes

Missing dependency on GeoModelKernel.

Frank Winklmeier authored 1 year ago and Johannes Elmsheuser committed 1 year ago

CondInputLoader: remove explicit semantics

C++ sets are now properly represented as python sets.

Beltran Fernandez Barbadillo authored 1 year ago and Johannes Elmsheuser committed 1 year ago

Add BeamSpot retriever to InDetRetrieversCfg (CA config)

Vakhtang Tsulaia authored 1 year ago and Johannes Elmsheuser committed 1 year ago

VP1Alg: fixed clang warning

Missing override keyword

Riccardo Maria Bianchi authored 1 year ago and Johannes Elmsheuser committed 1 year ago

VP1Algs migration to CA

Andrii Verbytskyi authored 1 year ago and Walter Lampl committed 1 year ago

Remove barcodes from the print statements and comments

Rename the Qt slots 'keyword' to Q_SLOTS.

Having this in the source confuses cppcheck, which reports syntax errors.
We can define slots as a macro expanding to nothing in the cppcheck
configuration, but if we do that globally, there are other packages that
break because they use slots as an identifier.  Using Q_SLOTS instead
prevents this name collision.

Uninitialized variables
Prefer range-for.

Rename the Qt slots 'keyword' to Q_SLOTS.

Having this in the source confuses cppcheck, which reports syntax errors.
We can define slots as a macro expanding to nothing in the cppcheck
configuration, but if we do that globally, there are other packages that
break because they use slots as an identifier.  Using Q_SLOTS instead
prevents this name collision.