Read mesh domains from vtk file 363

[ktbolt]

Finish #363.

I also cleaned up the interface for VtkData.h.

I don't think there are any tests for this.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 50 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.75%. Comparing base (8df8135) to head (26a622f).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
Code/Source/solver/VtkData.cpp	0.00%	26 Missing ⚠️
Code/Source/solver/vtk_xml.cpp	0.00%	17 Missing ⚠️
Code/Source/solver/read_msh.cpp	0.00%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #513      +/-   ##
==========================================
- Coverage   67.85%   67.75%   -0.10%     
==========================================
  Files         168      168              
  Lines       32749    32797      +48     
  Branches     5748     5766      +18     
==========================================
  Hits        22221    22221              
- Misses      10391    10439      +48     
  Partials      137      137              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[javijv4]

@ktbolt, I was looking into this code to reuse some of these functions for the 3D-0D cap implementation. However, I noticed that the functions VtkVtpData::has_cell_data and VtkVtuData::has_cell_data (lines 801 and 1060 in VtkData.cpp) define the number of arrays using GetPointData() instead of GetCellData(). For example:

int num_arrays = impl->vtk_polydata->GetPointData()->GetNumberOfArrays();

when it should be,

int num_arrays = impl->vtk_polydata->GetCellData()->GetNumberOfArrays();

I am not sure if I should open a new issue or something else.

[ktbolt]

@javijv4 Good catch !

This is a bug; open an Issue for it.