Ines Ochoa authored 6 years ago and Emmanuel Le Guirriec committed 6 years ago

Former-commit-id: 0d710ac44e3fc84937a732c9db36efc8fbb1eb67

Former-commit-id: c3a874bbdeaf2d9b0fc50d820978d6af43e02754

https://its.cern.ch/jira/browse/ATLEGAMDPD-95?focusedCommentId=2234324&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-2234324

Former-commit-id: 4a9088a18df39c694586af8000bbb7d069282f2d

Former-commit-id: 0594ef16838214e1351e1e058604687b675455b9

See above


Former-commit-id: 2a0fd3136f17269c298d0dcec8ae46306292d672

Former-commit-id: 25fccd2ace45b6e004d66e2e9bb343c4769e57f1

Former-commit-id: 34fbefab6b322d1f6da3c60195b956c66f35eba7

Jonathan Shlomi authored 6 years ago and Emmanuel Le Guirriec committed 6 years ago

Former-commit-id: 0f8efc75fbe5b6cf32e460cb56fd74ac873b4fe5

Former-commit-id: 31b81402e01175a1b01b977a1962a662c978f8b9

Former-commit-id: 43481cb3500ae8ed01d98c290624273f6fa692ec

Former-commit-id: 050b1ce6bf76655416a1baf2e07e2ba2cce0a4bd

Former-commit-id: 1b0bde0519857c4d0ea35778b9afd97ff6e6d18c

See above


Former-commit-id: 57c04a4c63fbae03d517ebda979bb9d53dc5d8ac

Former-commit-id: 2f8f930d9d6b0ff3c38b4fbcfb9bfd4b8eb5469e

Now the tool is more safe when reading the txt file and errors are
reported in case the file has problems. Also other minor changes:

   * Ignore first line of the txt file (since it is a header).
     Before it was included as a sample
   * Check if txt file exists
   * Check if there are problem when parsing the lines, skip in case
   * Check if file is empty
   * Use std::map instead of std::vector since the main operations
     are insertions and searches
   * Improved error message (remove context information, since
     it is added by the athena message service)
   * Parsing of the lines refactorized, much simpler, but same
     features
   * Remove non needed includes

Changes to the previous behaviour:

   * No more sample with dsid==0 corresponding to the header (not wanted)
   * Error messages in casae of: missing files, empty files, lines not
     in the desidered format
   * Due to the usage of std::map instead of std::vector, if there are
     duplicated samples, before the first won, now the last wins.
     The only case when I can see this to be legitimate is if one
     wants to overwrite a previous cross section (e.g. the default
     txt file with an analysis-based txt file), so "the last win"
     should be the desidered behaviour.


Former-commit-id: acdd06be5f260253ad56d472c3dcdd8f131ca055

Former-commit-id: 3aa3f673a97c6d072050ef5d3da5dabdd85283d2

Former-commit-id: 3ade871b45d42f5fcf762d51f520e21fcc3d63be

Allow a new option for people to configure the electron selection
tools using a configuration file instead of a pre-defined working
point. This commit also contains updates for the new jet uncertainty
recommendations, which obsoletes the JetResolution package


Former-commit-id: 817bfbadb6c058fd201ebb68ec5b866cab4e2661

Former-commit-id: 5704397fa0e9b090b0a6fd688e16f0773d205a43

Former-commit-id: 309d09ef263081af92f5c4230c589a40ff77154a

Small updates and fixes given the new August release of jet
uncertainties.  Also a small fix for the ECIDs accessor which had
the wrong type.


Former-commit-id: 37ecbbbd16866c5f029cabf138221f402d07ded4

Former-commit-id: 4c90b6b51c9c70a1d276385e88d8140367f7014c

Former-commit-id: b84d29ea83c5f925b885c377c0ea00115c0a4409

David Kirchmeier authored 6 years ago and Nils Erik Krumnack committed 6 years ago

Former-commit-id: e2ccbea461f6c0ad957e27305b8698f9f66dafe3

When ttree2hdf5 was passed and empty file would segfault. This commit
adds some protection against that situation. I also added a thin
wrapper on the `main(...)` function to catch logic errors and make the
error handling slightly cleaner.


Former-commit-id: 7bff2a1bb32a01992738e5d709136c21c9989535

Former-commit-id: a192d2842e908f37f213ab4555fa7133ecef6e55

Former-commit-id: 4748836730dd7786e02b545e398c9785a1ed9442

Former-commit-id: d3937e651d43d7e374803aa6d4119046930265d0

Former-commit-id: abc437714e1479fbbece643298f07151b025be38

Former-commit-id: e2c61f5980f9eb987ef755483285e71f23ddf79b

Former-commit-id: 3c51ab0662b38ab04afc10a9442bcbc9d6d3e7be

ATLASG-1450

It was noted that the derivation framework PrescaleTool is not safe in a multi-process/thread environment due to its use of an event counter, such that the order of execution needs to be the same to get reproducible results. This MR changes the tool to use the event number rather than a counter, and since the event number is independent of the the order of processing, the outcome of the prescaler is now exactly reproducible and predictable.


Former-commit-id: 34bb2bdbc7f28e74a954c5952ce0a7efb2fafc9b

Temporary fix/revert from previous commit, those 'cosmetic' changes turned out to have an actual change - moving generic track thinning first caused InDetTrackParticles to become much smaller for some so far mysterious reason


Former-commit-id: 6087b8257d4160c5bf94ecd291ebe6ef59e935d1

Former-commit-id: b57ffa978d4978969b9ba87efde282584e67d321

Former-commit-id: da2358828a2b9edc852d2d8845a1a6147a8e7433

Previous merge for b-tagging support in heavy ion jets (!12980) included
the aliasing of AntiKt4HI jet collection to AntiKt4EMTopo, which was not enough
(at least in the case of a test with 21.0) for the EM Topo jet reference data
to be recognized. Aliasing is now extended to AntiKt4TopoEM and AntiKt4LCTopo,
to solve this issue and be fully consistent with the scheme used for AntiKt4EMTopo.


Former-commit-id: ee0ae7a85c2794261c306c0a47cbfe2878e2cfc2

Former-commit-id: d4e1867397863bcbb834c536b5954c8112049f25

Former-commit-id: 02d268369c5a5e9962528cbf6417ac443ea0819e

Former-commit-id: ee146bd020affe69ca1b96f5f9b16e4db48b2fbe

Former-commit-id: e8fcbb1e277e6ef12dab1842e24e985be1523d9f