ads icon indicating copy to clipboard operation
ads copied to clipboard

Over-dependency on `httpready`

Open mjuric opened this issue 1 year ago • 1 comments

Hi,

This started as a bug report about the conda packaging (see here; Peter suggested I ping you to see if we can fix it upstream first.

The issue is that ads depends on httpready, but it seems that httpready is only being used for tests. Would it be possible to move httpready to an extras_require stanza, rather than general requirements section (i.e., to be installed only with pip install -e .[tests] or similar? That would allow keeping the runtime dependencies to a minimum (which conda cares about given the computationally-complex dependency resolution).

There's a bit more at https://github.com/conda-forge/ads-feedstock/issues/5 (and happy to provide more information if needed!).

mjuric avatar May 09 '24 20:05 mjuric

Hi,

Sorry for the delay. I was in the midst of moving overseas.

This seems reasonable, so I have made it so. Fixed with v0.12.6, which has been pushed to PyPI.

andycasey avatar May 21 '24 12:05 andycasey