uber / causalml

Uplift modeling and causal inference with machine learning algorithms
Other
4.87k stars 756 forks source link

maq git repo dependency blocks pypi publish #738

Closed jeongyoonlee closed 4 months ago

jeongyoonlee commented 4 months ago

Describe the bug

We can't publish a new version to PyPI as it doesn't allow publishing a package with git repo dependencies (ref: pypi/warehouse#9404).

To Reproduce

python -m twine upload dist/*

Expected behavior A new package is to be uploaded to PyPI without error.

Screenshots

WARNING  Error during upload. Retry with the --verbose option for more details.                                                                                                                                                                        
ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/                                                                                                                                                                               
         Invalid value for requires_dist. Error: Can't have direct dependency: 'maq @ git+https://github.com/grf-labs/maq.git@py0.2.2#egg=maq&subdirectory=python-package'      

Environment (please complete the following information):

Additional context

jeongyoonlee commented 4 months ago

@erikcs, do you have a plan to publish maq to PyPI?

erikcs commented 4 months ago

Hi @jeongyoonlee, sorry, I thought we touched potential dep issues here

Unfortunately, I don't currently have the bandwidth to put together a pypi source release, as making a sdist with C++ that installs/compiles w/o issues on all the possible system configurations out there can be challenging. It could risk making further complications for a causalML install.

git + pip was nice since that takes care of everything. Since that turns out not be viable option, what about the following:

Then:

ras44 commented 4 months ago

Just refreshed to see the updates from @erikcs addressing this issue 2 days ago. Feel free to ignore my recent pushes doing the same!