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

Update weaviate version in contextionary newspublication example #68

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

Conversation

dobraczka
Copy link

@dobraczka dobraczka commented Nov 12, 2024

Why:

This PR fixes: #67

What does your app/demo do?:

Loads the newspublication example using the contextionary vectorizer.

How Has This Been Tested?

I updated the docker-compose.yml using the code provided in the contextionary example
Then I started the updated docker-compose.yml and successfully ran the following Python code with version 4.9.3 of the Python client library:

import weaviate

with weaviate.connect_to_local() as client:
    articles = client.collections.get("Article")
    response = articles.query.hybrid(query="Hockey",limit=2)
    for o in response.objects:
        print(o.properties)

Have you included a README.md file in your example?

I have adapted the existing README to include the query example and updated outdated links.

@weaviate-git-bot
Copy link

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

@dobraczka
Copy link
Author

I agree with the CLA.

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.

Outdated weaviate version in contextionary newspublications example
2 participants