Clean up documentation build

[akaszynski]

This PR fixes a few annoying warnings that occur during our documentation build.

examples/01-filter/compute-volume.py

2022-05-31 20:51:29.891 (   0.661s) [        A417A000]         vtkDataSet.cxx:632   WARN| vtkUnstructuredGrid (0x3449120): Cell array RegionId with 1 components, has 553 tuples but there are only 518 cells
2022-05-31 20:51:29.929 (   0.699s) [        A417A000]         vtkDataSet.cxx:632   WARN| vtkUnstructuredGrid (0x3449120): Cell array RegionId with 1 components, has 553 tuples but there are only 518 cells

This is caused by a malformed regionID array when largest=True when using the connectivity filter. Since there's no reason to include this array in the first place when only extracting one region, simply remove it.

pyvista.examples.downloads.download_cgns_multi:

2022-05-30T22:45:01.7704924Z 2022-05-30 22:45:01.770 ( 410.807s) [        2EFFE740]      vtkCGNSReader.cxx:2079  WARN| vtkCGNSReader (0x55fe9a6272b0): Skipping BC_t node: BC_t type 'BCSymmetryPlane' not supported yet.
2022-05-30T22:45:01.7706086Z 2022-05-30 22:45:01.770 ( 410.807s) [        2EFFE740]      vtkCGNSReader.cxx:2079  WARN| vtkCGNSReader (0x55fe9a6272b0): Skipping BC_t node: BC_t type 'BCExtrapolate' not supported yet.
2022-05-30T22:45:01.7707238Z 2022-05-30 22:45:01.770 ( 410.807s) [        2EFFE740]      vtkCGNSReader.cxx:2079  WARN| vtkCGNSReader (0x55fe9a6272b0): Skipping BC_t node: BC_t type 'BCFarfield' not supported yet.
2022-05-30T22:45:01.7708011Z 2022-05-30 22:45:01.770 ( 410.807s) [        2EFFE740]      vtkCGNSReader.cxx:2079  WARN| vtkCGNSReader (0x55fe9a6272b0): Skipping BC_t node: BC_t type 'BCFarfield' not supported yet.

This is caused by loading the boundary patch load_boundary_patch. This has been disabled for this dataset.

Unnecessary file

The file my_theme.json is written when running doctests. The fix is to skip this test.

[codecov]

Codecov Report

Merging #2721 (50a35ff) into main (975645d) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2721      +/-   ##
==========================================
- Coverage   93.82%   93.77%   -0.06%     
==========================================
  Files          75       76       +1     
  Lines       16148    16171      +23     
==========================================
+ Hits        15151    15164      +13     
- Misses        997     1007      +10     

[adeak]

Looks good to me, thanks. It's unlikely that anyone would be relying on the broken RegionId to, I don't know, get the label of the largest region anyway, or get the number of regions, right? I guess they could just switch to largest=False and pick out the largest ones themselves.

[akaszynski]

Looks good to me, thanks. It's unlikely that anyone would be relying on the broken RegionId to, I don't know, get the label of the largest region anyway, or get the number of regions, right? I guess they could just switch to largest=False and pick out the largest ones themselves.

I'm reminded of

---

I think we're ok with removing this. regionID is undocumented (in our docs) anyway.