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

Add ML for 1.4.0 #64

Open
wants to merge 13 commits into
base: 1.4.0
Choose a base branch
from
Open

Add ML for 1.4.0 #64

wants to merge 13 commits into from

Conversation

DiegoPino
Copy link
Member

What?

Adds the (demo model driven) ML endpoints for the auth based/internal ML processing and KNN vector search

We will still provide the previous container for the many that don't need to evaluate ML

@DiegoPino DiegoPino self-assigned this Jun 24, 2024
@DiegoPino DiegoPino added this to the 1.4.0 milestone Jun 24, 2024
So (still testing... need to see this running) we used an virtual environment. If not all the previous calls end in something like

44.72 + python3 -m pip install --user -r requirements.txt --ignore-installed packaging
44.85 error: externally-managed-environment
44.85
44.85 × This environment is externally managed
44.85 ╰─>
44.85     The system-wide python installation should be maintained using the system
44.85     package manager (apk) only.
…einstalled

after the ML models

Might be that torch/pillow/pandas or scikit-learn end fetching their own requirements breaking everything (maybe it is Keras).

But this works. It does work (does it work?) pretty sure it does
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.

1 participant