Fix main failing due to sklearn bump

[ArnavBalyan]

Changes

• Main/CI is failing, sklearn got updated to 1.9.0rc1
• We are not pinned so we picked up the new release which has breaking changes
• Pin to 1.8.0 last safe version which keeps main intact

How I tested this

• CI

Notes

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

[ArnavBalyan]

cc @skrawcz thanks!