faces in xarray accessor exclude cyclic elements

[koldunovn]

Currently faces in xarray accessor - exclude cyclic elements. It's good for plotting, but not so nice for data analysis. We have to discuss what to keep. I would rather go for excluding cyclic elements during plotting itself, not to hide this info from the user.

[koldunovn]

Here is an example of a function that use all nodes:

pyfesom2/pyfesom2/regridding.py

Line 661 in a2951ce

def tonodes_jit(component, n2d, voltri, elem, e2d, lump2):

Anyway by cutting down the elements we don't provide the complete data to the user. I think whether or not cyclic points should be excluded should be decided on (plotting) function level, not at the dataset level.

[suvarchal]

it is fixed in #134 , but more work to go in that PR.