Skip to content

Conversation

@marcellodebernardi
Copy link
Contributor

In the newest library version sklearn was upgraded to 1.7.0, which was causing issues loading and using models from earlier versions of plexe.

Copilot AI review requested due to automatic review settings June 6, 2025 20:32
@marcellodebernardi marcellodebernardi added the bug Something isn't working label Jun 6, 2025
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Addresses backward compatibility issues by pinning scikit-learn version and adding deprecation support in pyproject.toml.

  • Pins scikit-learn to v1.6.1 to prevent breaking changes from v1.7.0 affecting existing models
  • Adds deprecated package v1.2.18 for better deprecation handling
  • Bumps package version from 0.26.0 to 0.26.2 for hotfix release

💡 (3/5) Reply to the bot's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This hotfix locks scikit-learn to version 1.6.1 for backwards compatibility, bumps the package version, and adds a new dependency.

  • Bumped project version from 0.26.0 to 0.26.2
  • Pinned scikit-learn exactly to 1.6.1
  • Added deprecated as a new dependency
Comments suppressed due to low confidence (2)

pyproject.toml:3

  • Consider updating CHANGELOG.md or release notes to document this hotfix release and explain the version bump.
version = "0.26.2"

pyproject.toml:55

  • Verify that the deprecated package is actually used in the codebase; if it's unused, removing it will keep dependencies lean.
deprecated = "^1.2.18"

marcellodebernardi and others added 3 commits June 6, 2025 13:34
# Conflicts:
#	poetry.lock
#	pyproject.toml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@marcellodebernardi marcellodebernardi merged commit 5497e2c into main Jun 6, 2025
5 checks passed
@marcellodebernardi marcellodebernardi deleted the fix/deprecated branch June 6, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants