- replace `.format` with f-strings
- replace `setattr` with direct `_properties` access in `merge`

it's not used for Python descriptor protocol and confuses people

Allow the comparison of properties following `SequenceSemantics` with
plain lists. Also remove the redundant `__ne__` operator which defaults
to `! __eq__` in Python3.

Closes #264.

Add a `clone` method to the Configurable class and tests.

The changes applied to the `merge` method
were made as a result of code profiling
and result in a roughly 20-30s improvement
in some ATLAS trigger jobs.

An new test was added in
`test_configurables_merge.py`
as a result of changes made to `merge` in order
to increase test coverage.

Apply the `OrderedSetSemantics` by default to `std::unordered_set` and
`set::set`. This fixes e.g. problems when merging the
ExtraInputs/Outputs properties of algorithms.

In line with how gaudi_add_tests(...) itself works.

`exec` changed from being a statement in Python2 to a function in
Python3. That also changes the formatting in `yapf`. Adjust remaining
occurences to follow the Python3 convention (no space before bracket).

Thomas Cluzel authored 4 years ago and Marco Clemencic committed 4 years ago

see https://its.cern.ch/jira/browse/LBCOMP-23



Co-authored-by: Marco Clemencic <marco.clemencic@cern.ch>

Walter Lampl authored 4 years ago and Marco Clemencic committed 4 years ago

patch generated by https://gitlab.cern.ch/wlampl/Gaudi/-/jobs/8165593

Do not call `__init__` during unpickling as that method may have been
overridden in derived classes. Instead set the properties directly.
Fixes issue #124.

Add unit test to test unpickling of derived configurable classes that
override the `__init__` method (issue #124).

Walter Lampl authored 4 years ago and Marco Clemencic committed 4 years ago

SequenceSemantics: Allow explicilty setting payload semantics, introduces __eq__ and __ne__ for _ListHelper

Gitlab CI authored 4 years ago and Walter Lampl committed 4 years ago

patch generated by https://gitlab.cern.ch/wlampl/Gaudi/-/jobs/8157139

Gitlab CI authored 4 years ago and Walter Lampl committed 4 years ago

patch generated by https://gitlab.cern.ch/wlampl/Gaudi/-/jobs/8114061

GaudiConfig2 semantics: Add update() method to _DictHelper. Main purpose is to set is_diry to true. Required to assign an empty dict to a property

Walter Lampl authored 5 years ago and Marco Clemencic committed 5 years ago

genconf: Use 'PublicToolHandle', 'PrivateToolHandle' and 'ServiceHandle' instead of GaudiHandleBase (same for handle-arrays)