-
Notifications
You must be signed in to change notification settings - Fork 53
Vector dtype recipe update for int8 #61
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
Vector dtype recipe update for int8 #61
Conversation
|
Realizing this actually depends on the upcoming release of redisvl. So we can hold it in a PR and merge / test once the package is available (no big deal). Make sure to update the README and colab links since you changed notebook name (which I agree with). Also is it always practical to inflate your integers using a scaled method before casting to int8? Aren't there models from hugging face or cohere we can showcase here for int8? |
|
There some non-scaled huggingface models, but the first I found still have typical values between (-5.0, +5) so using those raw still has a lot of rounding error when cast to int. |
|
In our test environment we're using the docker image redis-stack-server:latest, which still is on Redis 7.4. @tylerhutcherson we can update the test env, or wait until the latest docker image uses Redis 8.0 for int8 support. Thoughts? |
|
@justin-cechmanek let's update the CI to use the |
No description provided.