geowombat icon indicating copy to clipboard operation
geowombat copied to clipboard

fix: adding random forest classifier test which breaks

Open mmann1123 opened this issue 10 months ago • 1 comments

I identified an untested bug in geowombat. Basically sklearn-xarray needs scikit-learn=1.2.0 or it breaks with forest estimators. Adding test and trying to limit scikit version in setup.cfg for ml add-on install. had to set scikit-learn to 1.2.0 to fix the issue

What is this PR changing?

Add a description

Checklist

  • [ x] Remember to add a semantic tag to the commit name

Tag options:

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing or correcting existing tests
  • chore: Changes to the build process or auxiliary tools and libraries such as documentation generation

Example:

fix: <branch name> PR number

mmann1123 avatar Mar 28 '24 03:03 mmann1123