Install pyoorb from binary wheel

B612-Asteroid-Institute / precovery

Fast precovery of small body observations at scale

BSD 3-Clause "New" or "Revised" License

5 stars 2 forks source link

Install pyoorb from binary wheel #57

Closed spenczar closed 1 year ago

akoumjian commented 1 year ago

Obviously we also need to figure out why the tarball is not valid in the CI.

spenczar commented 1 year ago

What should the evolution from here be? Does it require getting both the .whl building PR and potentially our fork of pyoorb getting merged in? Or should we leave as is since we may be evaluating OD library usage long term.

Yes, I think we have two possible futures:

run a private PyPI with our packages, including modified forks, or
get our PR merged in and get oorb to publish to PyPI

spenczar commented 1 year ago

Obviously we also need to figure out why the tarball is not valid in the CI.

This is resolved, I expanded the shell variable the wrong way in the curl string. Added -f to the flags so it errors on a 404 (which this was) instead of downloading a 404 HTML page and trying to unzip it :]