Closed JoerivanEngelen closed 8 months ago
Fixes: https://github.com/Deltares/xugrid/issues/206, related to: https://github.com/Deltares/xugrid/issues/205
Unfortunately, I accidently pushed my changes to main. @Huite reviewed the changes via chat. This PR is to deal with these review comments.
main
PR implements the following:
__eq__
.identical
.equals
to_xr_if_possible()
Fixes: https://github.com/Deltares/xugrid/issues/206, related to: https://github.com/Deltares/xugrid/issues/205
Unfortunately, I accidently pushed my changes to
main
. @Huite reviewed the changes via chat. This PR is to deal with these review comments.PR implements the following:
__eq__
can be removed, as it is not equivalent to the pandas behavior.identical
can be renamed to.equals
and original.equals
can be removed.equals
, to make it more comprehensibleto_xr_if_possible()
is removed as it was unnecessary