Algorithm/tool caching and removal of hashes in names

• introduced class stores for tools/algorithms to have earlier deduplication
• same conf, different names -> failure
• fixed some bugs
• more checks, you cannot give plain Configurables as inputs to an Algorithm/Tool constructor
• formatting
• added unique names without hashes

closes #30 (closed)
closes #12 (closed)
closes #31 (closed)
closes #26 (closed)

added 1 commit

• ea5185c7 - Fixed formatting

Compare with previous version

changed the description

resolved all discussions

added 1 commit

• 62582667 - update error message

Compare with previous version

resolved all discussions

The Python formatter is pretty bad... otherwise LGTM

added 1 commit

• 17b4099a - names without hashes

Compare with previous version

changed the description

i think after this MR we can start moving to LHCb, do you agree?

changed title from WIP algorithm/tool caching to WIP algorithm/tool caching, removal of hashes in names and fail in case of different names for same configuration

Pretty much. I would like to sweep through PyConf for formatting, to make sure the names are all good, and to add any missing docs. Do you want any other issues addressed before moving @nnolte @rmatev?

assigned to @rmatev

resolved all discussions

the test_hlt2_example test fails for me on this branch

Traceback (most recent call last):
  File "<string>", line 25, in <module>
  File "/home/rmatev/distcc/stack/LHCb/InstallArea/x86_64-centos7-gcc8-opt/python/LoKiCore/decorators.py", line 3020, in hybrid_context_deco
    format(symbol_name, ' or '.join(context_types)))
AssertionError: Functor AALLSAMEBPV requires context (type dvalgo) but none is defined.

added 1 commit

• 74c41252 - fix tool deduplication bug

Compare with previous version

resolved all discussions