LoadMask Reports Misleading Error For Non-Conform DetID List

[Peter Parker]

A mask file with a trailing comma results in an error of

Pixel w/ ID = 0 Cannot Be Located

being reported to the results log. Currently the fix is to change this:

<?xml version="1.0"?>
<detector-masking>
        <group>
                <detids>10,</detids>
        </group>
</detector-masking>

to this:

<?xml version="1.0"?>
<detector-masking>
        <group>
                <detids>10</detids>
        </group>
</detector-masking>

But it would be nice to clear away this edge case when parsing the IDs.

[Stuart Campbell]

This ticket has been transferred to github issue ​10589