Instead of create_dolfinx_function_with_point_data(...) and create_dolfinx_function_with_cell_data I think it would be a lot simpler to have .read_point_data() and .read_cell_data() which is the convention io4dolfinx uses.
"create dolfinx function` is a bit redundant since we already know this is foam2dolfinx
Instead of
create_dolfinx_function_with_point_data(...)andcreate_dolfinx_function_with_cell_dataI think it would be a lot simpler to have.read_point_data()and.read_cell_data()which is the conventionio4dolfinxuses."create dolfinx function` is a bit redundant since we already know this is foam2dolfinx