Update error messages to be more descriptive.

@pisarna @tbold @ponyisi

I've updated a few of the error messages.

1. In AthMonitorAlgorithm, the error getGroup function told you which group name was missing, but now it also tells you from which algorithm (in case there are multiple instances of the same algorithm in a single job.)

2. In HistogramDef, warns the user that the parsing error may have been caused by a bad use of delimiters. One user was using commas in the title, and had trouble since he presumably thought that the error was casting float to int, or something like that. If there's a better way to do this one, I can certainly try it.

Chuck

unmarked as a Work In Progress

changed the description

Changed from WIP to "require approvals" so the unit tests will run.

This merge request affects 1 package:

• Control/AthenaMonitoring

Adding @ssnyder ,@rbianchi as watchers

added Core DQ master review-pending-level-1 labels

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-24764-2019-07-10-01-38
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 1028]

The discussions need to be resolved before it can be approved. I am changing the labels until its done. -Ruchi (L1)

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 61447858 - Add list of available groups in FATAL message

Compare with previous version

resolved all discussions

This merge request affects 1 package:

• Control/AthenaMonitoring

Adding @ssnyder ,@rbianchi as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-24764-2019-07-12-23-55
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 1120]

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit d3f436cd

added sweep:ignore label