We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
[optional format]
Identify the file to be fixed Semantic_text_search_using_embeddings.ipynb
Describe the problem
from utils.embeddings_utils import get_embedding, cosine_similarity
This import line is failing in the cookbook. Not sure if it's intended as a relative import
Describe a solution A clear and concise description of what a fixed version should do.
For cosine_similarity, we could use scipy tools:
cosine_similarity
from scipy.spatial import distance distance.cosine()
For get_embedding, we could write it out for clarity:
get_embedding
def get_embedding(text, model): ...
Screenshots
Additional context General suggestion here is to remove the relative importing so that people can get to "Hello World" faster here
The text was updated successfully, but these errors were encountered:
No branches or pull requests
[optional format]
Identify the file to be fixed
Semantic_text_search_using_embeddings.ipynb
Describe the problem
This import line is failing in the cookbook. Not sure if it's intended as a relative import
Describe a solution
A clear and concise description of what a fixed version should do.
For
cosine_similarity
, we could use scipy tools:For
get_embedding
, we could write it out for clarity:Screenshots
Additional context
General suggestion here is to remove the relative importing so that people can get to "Hello World" faster here
The text was updated successfully, but these errors were encountered: