maxdotio / ai-search-training

AI Powered Search Training! These are the labs to go with our comprehensive training course.
https://max.io/private-training.html
1 stars 1 forks source link

Upgrading Requirements.txt causes breaking changes. #1

Closed thedayisntgray closed 4 days ago

thedayisntgray commented 4 days ago

Hey @binarymax ,

Just a heads up. I noticed that upgrading the requirements.txt via this pr creates a breaking change. Specifically the issue I see is with the sentence-transformers library. We either need to downgrade that one gem or rework the examples to use the correct import method.

I'll plan to take a look further look sometime this weekend.

But I did confirm that reverting the requirements.txt file fixes this issue for sentence-transformers. Screenshot 2024-11-30 at 11 25 50 AM

binarymax commented 4 days ago

Hey! Yeah I ran into that exact same issue. You'll need to rebuild the entire docker for this to work:

docker compose build

It's fine to keep your Ruby port the same.

thedayisntgray commented 4 days ago

You're are right! Closing this.