MannLabs / alphadia

modular & open DIA search
https://alphadia.readthedocs.io
Apache License 2.0
41 stars 3 forks source link

Docker Container Missing MANIFEST.in #261

Closed dnusinow closed 2 months ago

dnusinow commented 2 months ago

Thank you for alphadia and congratulations on the release!

Describe the bug The alphadia docker container errors with the following error:

ERROR: [Errno 2] No such file or directory: '/usr/local/lib/python3.9/site-packages/alphadia/constants/default.yaml'.

This was fixed by adding COPY MANIFEST.in MANIFEST.in to the Dockerfile around line 40.

To Reproduce Steps to reproduce the behavior: I followed the instructions here which worked perfectly with the patch suggested above.

Expected behavior The alphadia run should go to completion.

Logs I don't have the log anymore after patching the dockerfile.

Version (please complete the following information):

GeorgWa commented 2 months ago

Hi, thanks for reporting the Bug! Could you open a PR with the suggested change against the dev branch?

dnusinow commented 2 months ago

No problem, done!

On Mon, Jul 1, 2024 at 10:35 AM Georg Wallmann @.***> wrote:

Hi, thanks for reporting the Bug! Could you open a PR with the suggested change against the dev branch?

— Reply to this email directly, view it on GitHub https://github.com/MannLabs/alphadia/issues/261#issuecomment-2200334727, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYWTMKZZSFXZNNJG4G7DXLZKFSMJAVCNFSM6AAAAABKFWMXD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBQGMZTINZSG4 . You are receiving this because you authored the thread.Message ID: @.***>