added check to ensure readgroup ID uniqueness within all BAMs processed together, usually all BAMs belong to the same donor. also changed regex for the PU field which allows run name to have underscores in them