Skip to content

Commit bbac850

Browse files
committed
refactor: do not export legacyVectors directly
1 parent 9f2a536 commit bbac850

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

src/collections/configure/index.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import {
1515
import generative from './generative.js';
1616
import reranker from './reranker.js';
1717
import { configure as configureVectorIndex, reconfigure as reconfigureVectorIndex } from './vectorIndex.js';
18-
import { legacyVectors, multiVectors, vectors } from './vectorizer.js';
18+
import { multiVectors, vectorizer, vectors } from './vectorizer.js';
1919

2020
import { parseWithDefault } from './parsing.js';
2121

@@ -61,9 +61,9 @@ const configure = {
6161
multiVectors,
6262
reranker,
6363
/**
64-
* @deprecated Use `vectors` instead.
64+
* @deprecated Use `configure.vectors` instead.
6565
*/
66-
vectorizer: legacyVectors,
66+
vectorizer: vectorizer,
6767
vectors,
6868
vectorIndex: configureVectorIndex,
6969
dataType,
@@ -311,6 +311,6 @@ export {
311311
tokenization,
312312
vectorDistances,
313313
configureVectorIndex as vectorIndex,
314-
legacyVectors as vectorizer,
314+
vectorizer,
315315
vectors,
316316
};

src/collections/configure/vectorizer.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,10 +117,8 @@ const formatMulti2VecFields = (
117117
/** Previously all text-based vectorizers accepted `vectorizeCollectionName` parameter, which was meaningless for some modules and caused others to produce confusing results (see details below). Moving forward, we want to deprecate the usage of this parameter.
118118
*
119119
* Collections with `vectorizeCollectionName: true` generate embeddings even if they have no vectorizeable properties. This means all generated embeddings would embed the collection name itself, which makes them rather meaningless.
120-
*
121-
* @deprecated Use `vectors` instead.
122-
* */
123-
export const legacyVectors = {
120+
*/
121+
const legacyVectors = {
124122
/**
125123
* Create a `VectorConfigCreate` object with the vectorizer set to `'none'`.
126124
*

0 commit comments

Comments
 (0)