The map_attributes_to_num() gets double called

[NickChapman]

At the start of fill_holes() it calls map_attributes_to_num(). This already gets called in the init so potentially we don't need to rebuild that mapping It might be better to check if the mapping is built rather than double call.