probability icon indicating copy to clipboard operation
probability copied to clipboard

Publish SDist of tensorflow-probability to PyPi

Open Saadorj opened this issue 2 years ago • 3 comments

On pypi, the SDist of tensorflow-probability is not published. Only a binary artifact is published. Is there a reason for not publishing SDist (.tar.gz) along the bdist_wheel that is currently published?

If the package is not platform-dependent, then perhaps it is possible to publish sdist? If so, would it be possible to publish it for versions 0.15.0+ on PyPi?

Thank you!

Saadorj avatar Aug 22 '22 19:08 Saadorj

Here is a PR to hopefully resolve this issue. https://github.com/tensorflow/probability/pull/1687

hgonzale-li avatar Feb 23 '23 19:02 hgonzale-li

Thanks, both. What is the benefit to having a source tarball included? I can easily find the installed sources in, say, a virtualenv lib directory. Does this help some other set of tools/IDEs or something?

csuter avatar Feb 23 '23 20:02 csuter

It helps our internal repository system to automatically accept and upgrade to the latest TF-probability version. Without the sdist tarball we have to do this work manually for each new version.

Most TF repos already distribute sdist tarballs, and at the moment we only observe this issue in the TF-probability package.

hgonzale-li avatar Feb 23 '23 22:02 hgonzale-li