linkedin / linkedin/greykite

Greykite pins dependencies to old versions and cannot be installed with other packages in general (or not easily)

Open
#143 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
1.9k
Forks
107
PR merge metrics
No merged PRs in 30d

Description

The `greykite` package - as released on `pypi` pins all its dependencies to exact versions or narrow version ranges.

This is not a good idea, as this makes installation together with other packages very difficult, and impossibly for any other package with a comparably narrow, but different version range for one of the dependencies in question.

A number of these are, in addition, very outdated versions of packages, e.g., `pandas 1.X` (now on 2.2) or `scikit-learn 1.3.1` (now on 1.6) - `pandas` may be sensible due to some breaking changes in v2, but there should not be a need ot pin `scikit-learn==1.3.1`.

I would suggest to adopt larger possible ranges for the dependencies.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no files or tests. Start by locating Greykite's package dependency metadata and reviewing the current pins, then test installation alongside packages with compatible but different requirements; done means dependency ranges are broadened without breaking supported installs.

Written by the indexing model from the issue text.

Assessment

Tech stack
pandas, python, scikit-learn
Domain
build-system, tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.