search

nextstrain / ncov-ingest

A pipeline that ingests SARS-CoV-2 (i.e. nCoV) data from GISAID and Genbank, transforms it, stores it on S3, and triggers Nextstrain nCoV rebuilds.
MIT License
36 stars 20 forks source link

Unable to build Docker image #413

Closed victorlin closed 7 months ago

victorlin commented 1 year ago

I ran update-image.yml on master and it failed while running pipenv sync in the Docker build.

ValueError: "Requirement" object has no field "use_pep517"

Others on the internet have reported that this error comes from a version of PyYAML (which is included in the Pipfile.lock).

I tried re-generating the lock file using pipenv lock locally but it failed.

tsibley commented 1 year ago

I haven't dug, so this is a guess, but I'm guessing this issue is restricted to Pipenv and the same version of PyYAML would be fine with standard Pip. If so, that's all the more reason to ditch Pipenv, like we've done elsewhere. If we need strict locking, we can use pip-compile from pip-tools, as elsewhere.