Scott Snyder authored 5 years ago and Walter Lampl committed 5 years ago

Add option to printConfig to allow restricting the set of components
to be printed.

Sometimes, i have observed jobs using new configuration failing on a call to Gaudi
with the error that gbl.Gaudi is not a Gaudi instance.  This can be avoided
by touching the pyroot object earlier.

Remove ForceRead

Tomasz Bold authored 5 years ago and Vakhtang Tsulaia committed 5 years ago

made sure that the top main servvices CA is never merged to another accumulator but the others are merged to it

Edward Moyse authored 5 years ago and Frank Winklmeier committed 5 years ago

Fix handling of single private tools, and reverse order of primary component determination, to favour private tools over services.

This latter change is because you can get a service from another CA Cfg method, but if you explicitly set a private tool, that must be the intended primary component (since private tools cannot be left dangling).

Scott Snyder authored 5 years ago and Walter Lampl committed 5 years ago

Strings passed to BootstrapHelper should be encoded strings, not unicode objects.

Tim Martin authored 5 years ago and Walter Lampl committed 5 years ago

Modern CA configuration inside newJOtest.py -> TriggerConfig.py
Old random-python-everywhere configuration inside full_menu.py

Fix warnings/errors flagged by `flake8`. Mostly unused variables/imports
or stylistic code problems.

On kernels with Yama enabled, ptrace may not work by default on processes
which are not decendants of the tracing process.  Among other things, that
causes the way we attach the debugger to fail.  However, we can disable this
on a per-process basis.  So do that when we initialize Athena.

See https://www.kernel.org/doc/Documentation/security/Yama.txt and prctl(2).

In order for an algorithm implemented in python to function, setup() must have
been called on its Configurable during initialize().  In the old configuration,
the Atlas AppMgr in AthenaCommon does this for all Configurables.  But with the
new configuration, we're using the Gaudi AppMgr directly, which does not
do this.  So in createApp, test to see if we're adding any python algorithms,
and if so, call setup() on them directly.

When doing this, be careful not to fail if AthenaPython is not available.

In printProperties(), print the properties in sorted order.

 - Fix flag testing in python 3.
 - Add option to loadAllDynamicFlags to suppress the annoying output.
 - Fix a typo in an error message.
 - Break out the conditions algorithm printing from printConfig
   into a separate function, so that we can request printing just that
   part of the configuration.  Allow including information on properties.
   

Fix references to func_name/func_code for python3.

Initial python3 port.  Not intended to be complete yet; only that required
for basic athena/configuration jobs.

- use lazy string formatting in logging calls
- some Python3 fixes
- remove print statements

Emmanuel Le Guirriec authored 5 years ago and Adam Edward Barton committed 5 years ago

Make tools private in
- InDetDetailedTrackSelectorTool.cxx
- TrackSummaryTool.cxx
- KalmanVertexOnJetAxisSmoother.cxx
- TrkVKalVrtFitter.cxx

Create Cfg methods for new config

Fix a typo in the documentation.

ComponentAccumulator: Introduce primary component instead of returning tuples. Adjusting (some) of the existing new-style configurations

ComponentAccumulator: Add doc-strings for setPrivateTool and popPrivateTool, move check for dangling private tool(s) to __del__