Because 'sites' and 'id_by_site' are required for backwards
compatibility they must know about the sites of the previous unit cell.
In principle we could compute these as-needed, but explicitly storing
them allows the implementations of _Site and _IdBySite to remain simple.