If a sequence dictionary is provided (which seems to be provided in the constructor), then shouldn't we be able to get the contigId from the chr name and the sequence dictionary? That might be a bit cleaner?