Is there a reason why we use `data_values` here, and not `values`?