search

tensorflow / probability

Probabilistic reasoning and statistical analysis in TensorFlow
https://www.tensorflow.org/probability/
Apache License 2.0
4.27k stars 1.11k forks source link

Publish SDist of tensorflow-probability to PyPi #1605

Open Saadorj opened 2 years ago

Saadorj commented 2 years ago

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!

hgonzale-li commented 1 year ago

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

csuter commented 1 year ago

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?

hgonzale-li commented 1 year ago

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.