Skip to content

Conversation

LaoGunz
Copy link
Collaborator

@LaoGunz LaoGunz commented Sep 3, 2022

New logic for compose env

class ComposeEnv:

def __init__(self):
self.__resource = ComposeExecutor()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This

def has_collection(name_collect: str = "hello_milvus"):
has = utility.has_collection(name_collect)

return f"Does collection hello_milvus exist in Milvus: {has}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add list() collections

rng.random((num_entities, dim)), # field embeddings, supports numpy.ndarray and list
]
vectors_to_search = entities[-1][-2:]
search_params = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add this as kwargs

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as json

@LaoGunz LaoGunz changed the title Venv logic Docker Compose (fastapi, milvus) logic Sep 3, 2022
@sokolegg
Copy link
Member

sokolegg commented Sep 9, 2022

Approved! Nice work

@LaoGunz LaoGunz merged commit 479de2a into main Sep 9, 2022
sokolegg pushed a commit that referenced this pull request Dec 22, 2022
Docker Compose (fastapi, milvus) logic
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.

3 participants