extended subgrid functionality

[chmerdon]

extended subgrid features:

• new argument 'support' which can be ON_CELLS (default subgrid behaviour as before), ON_BFACES (default subgrid behaviour with boundary = true which also still works) and ON_FACES (new behaviour that generates a subgrid with faces)
• SubGrid type has now a parameter that knows the support relative to parent grid, removed BoundarySubGrid type
• removed get_facegrid, get_bfacegrid, get_edgegrid in derived.jl (that offer no new functionality compared to subgrid)

[j-fu]

IMHO assemblytypes.jl needs to be included before subgrids.jl.

Generally, formally removing API (including keyword args) is a breaking change.
Unless we are really sure that nobody else uses this (I think I use boundary=true) we should keep the old API, possibly with a deprecation warning.

[chmerdon]

Yes, I am on it. Unfurtunately also som subgrid tests don't run through ^^

[chmerdon]

Ok, now it should work... I messed up the old CoordinateSystem handling for codimension -1 grids

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 55.55556% with 8 lines in your changes missing coverage. Please review.

Project coverage is 74.24%. Comparing base (06a2f11) to head (2eabbea).
Report is 43 commits behind head on master.

Files	Patch %	Lines
src/subgrid.jl	52.94%	8 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
+ Coverage   73.01%   74.24%   +1.23%     
==========================================
  Files          27       32       +5     
  Lines        4239     4589     +350     
==========================================
+ Hits         3095     3407     +312     
- Misses       1144     1182      +38     

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

[j-fu]

One bug where based is still used.

[chmerdon]

You are right. I fixed it the two 'based' occurences. No API was removed, boundary = true should still work as before and is also covered by the (same) tests.

[j-fu]

Just checked with VoronoiFVM, tests are ok.