Skip to content
Snippets Groups Projects

adding more details to AsgElectronLikelihoodTool initialization failure error messages

Merged adding more details to AsgElectronLikelihoodTool initialization failure error messages
(removed):master-ELHT-160621-for_debug into master
Merged Julien Maurer requested to merge (removed):master-ELHT-160621-for_debug into master

In order to investigate ATLASRECTS-6418: adding as suggested by @christos a bit more information (paths to the specified and resolved config and PDF files) to the error messages printed by the AsgElectronLikelihoodTool when initialization fails. Also including a couple trivial checks and printing warnings accordingly (e.g. when the default PDF file name is used, which should not really happen). The tool's behaviour should be the same as before with one exception: the initialization will fail early if the PDF filename is empty (but previously that would probably have caused a failure soon afterwards anyway). In principle, for normal usage there should be no more printouts in the log than currently.

Merge request reports

Approval is optional

Merged by Frank WinklmeierFrank Winklmeier 3 years ago (Jun 17, 2021 9:12am UTC)

Merge details

  • Changes merged into master with 557b10a4 (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply