Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/use uv instead of poetry #4

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

drumtorben
Copy link

@drumtorben drumtorben commented Dec 27, 2024

Closes #3

Changes

Switch everything to UV

Reasons

Testing

Impacts

Notes

@pazevedo-hyland
Copy link
Collaborator

Hey @drumtorben thanks for this PR.
Are you able to run all the pipelines in your local with this branch? I couldn't, I had to fix the dependencies I had on the poetry lock to the uv lock to make it work.

have you tested all automations? inv formats etc..?
image

I'm not sure Im using uv correctly :P maybe I missed something. Once I get all pipelines working on this branch we can merge it!

@drumtorben
Copy link
Author

drumtorben commented Dec 29, 2024

Both is working for me. I get the MLFlow error in the end, but that was the same for me before the switch.
Bildschirmfoto 2024-12-29 um 17 02 29

Yeah, from the screenshot it could probably be that you are in the wrong environment or so? I am not completely sure about that ...

EDIT: Fixed the MLFlow error by setting AWS_ACCESS_KEY_ID, AWS_REGION, and AWS_SECRET_ACCESS_KEY in the .env to dummy values. I now get a connection error, but this is of course due to the dummy values, as I have no AWS instances running.

@drumtorben
Copy link
Author

One thing I was just thinking about:
Did you delete your .venv folder, which was created by poetry? If not, delete it and then run uv sync —all-groups. This creates then a new .venv folder.

If you are in VSCode you now should be asked about using this environment, else you have to activate it manually with source .venv/bin/activate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Use UV instead of Poetry
2 participants