`uv` installation failure: `pyro-ppl` `test` dependency uses non-compliant URL extension

[Qazalbash]

When attempting to install pyro-api using uv, the process fails during metadata parsing. This is because uv requires direct URL dependencies to have a supported file extension (e.g., .tar.gz, .zip).

The current setup.py points to a GitHub API tarball endpoint, which lacks an extension: https://api.github.com/repos/pyro-ppl/pyro/tarball/dev

You can reproduce it as,

uv pip install https://github.com/pyro-ppl/pyro-api/archive/master.zip

error: Failed to parse metadata from built wheel
  Caused by: Expected direct URL (`https://api.github.com/repos/pyro-ppl/pyro/tarball/dev`) to end in a supported file extension: `.whl`, `.tar.gz`, `.zip`, `.tar.bz2`, `.tar.lz`, `.tar.lzma`, `.tar.xz`, `.tar.zst`, `.tar`, `.tbz`, `.tgz`, `.tlz`, or `.txz`
pyro-ppl @ https://api.github.com/repos/pyro-ppl/pyro/tarball/dev ; extra == "test"
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

[fehiepsi]

I'm not familiar with uv but can we use

'pyro-ppl @ https://github.com/pyro-ppl/pyro/archive/dev.tar.gz',
'numpyro @ https://github.com/pyro-ppl/numpyro/archive/master.tar.gz',
'funsor @ https://github.com/pyro-ppl/funsor/archive/master.tar.gz',

instead?

[Qazalbash]

We think this will not resolve the problem, because it is coming from executing this line.

https://github.com/Qazalbash/numpyro/blob/8c1b1b07cc2ae54c1bf025910f20b8e484f65f8a/.github/workflows/ci.yml#L77

[fehiepsi]

I'm confused. The error is

Caused by: Expected direct URL (`https://api.github.com/repos/pyro-ppl/pyro/tarball/dev`) to end in a supported file extension: `.whl`, `.tar.gz`, `.zip`, `.tar.bz2`, `.tar.lz`, `.tar.lzma`, `.tar.xz`, `.tar.zst`, `.tar`, `.tbz`, `.tgz`, `.tlz`, or `.txz`

and those links end with .tar.gz.

[Qazalbash]

When I try to install pyro-ppl/pyro-api using uv, it also installs the test dependencies, which use a non-standard dependency format that worked with pip but not with uv.

pyro-api/setup.py

Lines 40 to 42 in 04b2e88

	'pyro-ppl@https://api.github.com/repos/pyro-ppl/pyro/tarball/dev',
	'numpyro@https://api.github.com/repos/pyro-ppl/numpyro/tarball/master',
	'funsor@https://api.github.com/repos/pyro-ppl/funsor/tarball/master',

[fehiepsi]

I'm not familiar with uv but can we use

'pyro-ppl @ https://github.com/pyro-ppl/pyro/archive/dev.tar.gz',
'numpyro @ https://github.com/pyro-ppl/numpyro/archive/master.tar.gz',
'funsor @ https://github.com/pyro-ppl/funsor/archive/master.tar.gz',

instead?

I see that such format causes the error. But would this tar.gz format also cause the issue?

[fehiepsi]

I meant this change

- 'pyro-ppl@https://api.github.com/repos/pyro-ppl/pyro/tarball/dev', 
- 'numpyro@https://api.github.com/repos/pyro-ppl/numpyro/tarball/master', 
- 'funsor@https://api.github.com/repos/pyro-ppl/funsor/tarball/master', 
+ 'pyro-ppl @ https://github.com/pyro-ppl/pyro/archive/dev.tar.gz',
+ 'numpyro @ https://github.com/pyro-ppl/numpyro/archive/master.tar.gz',
+ 'funsor @ https://github.com/pyro-ppl/funsor/archive/master.tar.gz',

[Qazalbash]

I meant this change

• 'pyro-ppl@https://api.github.com/repos/pyro-ppl/pyro/tarball/dev',
• 'numpyro@https://api.github.com/repos/pyro-ppl/numpyro/tarball/master',
• 'funsor@https://api.github.com/repos/pyro-ppl/funsor/tarball/master',

• 'pyro-ppl @ https://github.com/pyro-ppl/pyro/archive/dev.tar.gz',
• 'numpyro @ https://github.com/pyro-ppl/numpyro/archive/master.tar.gz',
• 'funsor @ https://github.com/pyro-ppl/funsor/archive/master.tar.gz',

These changes are working as expected. Do you want me to create a PR?

[fehiepsi]

Yes, please.

[Qazalbash]

Please see #26