Can we adjust the naming of these to 'apply_givens_in_place', 'merge_R_on_Q', 'merge_QR_up', etc?