Pass metric type to vector search function

Thanks Josh; I think we should probably pass something more generic to avoid further interface changes in the future though. If for example we pass the QueryInterface you can then access the query, index, and server. So in your case I believe $query->getIndex()->getServerInstance()->getBackendConfig() should contain the metric.

Also latest phpcs changes use statement order, so test coverage is failing.

@joshhytr this issue was flagged is important for [Meta] Bring AI Search out of Experimental 🌱 [Meta] Bring AI Search, Assistants and Chatbot out of Experimental Active . We would like to have it fixed until 4th February 2025. Will you manage or are you happy for maintainers to help?

Thanks @seogow, thanks @scott_euser.

I've updated the MR with the suggested change and have been able to obtain the metric_type from the query as required.

@seogow or @scott_euser, could you check this out?

This seems fine yeah, we just need to do a coordinated release with Milvus and Pinecone as the interface will cause a breaking change. Are there any other VDB providers we know of?

I believe we then need to add:

1. conflict to composer.json of AI module for current release of Milvus and Pinecone
2. conflict to composer.json of Milvus and Pinecone for current release of AI module

That way we force people to composer update them together. That sound right?

@scott_euser in #8 is correct.

This does break https://www.drupal.org/project/ai_vdb_provider_milvus: →

"PHP message: PHP Fatal error: Declaration of Drupal\ai_vdb_provider_milvus\Plugin\VdbProvider\MilvusProvider::vectorSearch(string $collection_name, array $vector_input, array $output_fields, mixed $filters = '', int $limit = 10, int $offset = 0, string $database = 'default'): array must be compatible with Drupal\ai\AiVdbProviderInterface::vectorSearch(string $collection_name, array $vector_input, array $output_fields, Drupal\search_api\Query\QueryInterface $query, string $filters = '', int $limit = 10, int $offset = 0, string $database = 'default'): array in drupal-10.4.x/html/modules/contrib/ai_vdb_provider_milvus/src/Plugin/VdbProvider/MilvusProvider.php on line 597

Also 🐛 PHP Fatal error: Declaration of Drupal\ai_vdb_provider_postgres\Plugin\VdbProvider\PostgresProvider::vectorSearch Active

Temporary patch for fix them (i make this for my test)

Thanks @joshtyr - this is going into 1.1.x-dev, that will have this breaking change. We will update the Milvus and Pinecone providers to also have 1.1.x releases fixed to the 1.1.x release of AI module.

This means that you can have a requirement to 1.1.0-dev for now and later 1.1.0 and hopefully the vdb provider should work as expected.

🐛 Update to use ai 1.1.x-dev Active

Automatically closed - issue fixed for 2 weeks with no activity.