Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish SDist of tensorflow-probability to PyPi #1605

Open
Saadorj opened this issue Aug 22, 2022 · 3 comments
Open

Publish SDist of tensorflow-probability to PyPi #1605

Saadorj opened this issue Aug 22, 2022 · 3 comments

Comments

@Saadorj
Copy link

Saadorj commented Aug 22, 2022

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
Copy link

Here is a PR to hopefully resolve this issue.
#1687

@csuter
Copy link
Member

csuter commented Feb 23, 2023

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
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants