tensorflow / tensorflow/probability

inconsistent Requires-Dist leads to ImportError with 0.25

Open
#1,854 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Jupyter Notebook
Stars
4.4k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

5fa62e16b7859d38afb427e45ffbe03959526e75 set a runtime check with required_tensorflow_version = '2.18', but https://github.com/tensorflow/probability/blob/v0.25.0/setup.py#L52 still has the Requires-Dist set to 2.16.

This will lead to an import error like:

E   ImportError: This version of TensorFlow Probability requires TensorFlow version >= 2.18; Detected an installation of version 2.16.2. Please upgrade TensorFlow to proceed.

which could be avoided if the Requires-Dist matched.

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

Start in setup.py at the Requires-Dist entry for TensorFlow and compare it with the required_tensorflow_version runtime check mentioned in the issue. Done means the package metadata and runtime requirement agree, and an installation check no longer permits TensorFlow 2.16 with version 0.25.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, tensorflow
Domain
build-system, machine-learning
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.