Q: geocat.comp.ndpolyfit() moved upstream to Xarray. Should ndpolyfit be removed from geocat.comp?

[pilotchute]

Xarray implemented polyfit and polyval on: pydata/xarray#3733
Should the duplicate functionality in geocat.comp.ndpolyfit be preserved?

To start the discussions:
Pros: maintains backward compatibility for all projects.

Cons: functionally with current polyfit in pydata.xarray is duplicated, and pydata.polyfit should become the reference implementation now that it has been moved upstream.

Potential resolution: refactor geocat.comp.ndpolyfit as an alias to pydata.xarrray.polyfit. this preserves compatibility with code that calls comp.ndpolyfit, while allowing xarray.polyfit to be the implementation.

[pilotchute]

• Resolution: refactor geocat.comp.ndpolyfit as an alias to pydata.xarrray.polyfit. this preserves compatibility with code that calls comp.ndpolyfit, while allowing xarray.polyfit to be the implementation.
• tag with branch once this imporovement is started.

[anissa111]

@pilotchute any updated thoughts on this?