lemma-osu / sknnr-spatial

https://sknnr-spatial.readthedocs.io
0 stars 0 forks source link

Implement estimator wrapper #13

Closed aazuspan closed 4 months ago

aazuspan commented 5 months ago

As discussed in #12, wrapping sklearn estimators in a custom class would allow us to:

  1. Track the data used for fitting, e.g. the number and names of targets and whether single-output targets are squeezed.
  2. ~Check fitted feature names against image feature names and raise appropriate warnings (currently we always get warnings when fitting with dataframes, but we can potentially suppress that when predicting to named DataArrays and Datasets).~ Moving this detail to #17.
  3. Only override relevant methods, e.g. kneighbors for kNN estimators.