Error when passing show_progress_bar as a parameter for HFTextVectorizer

[raphaeldelio]

When creating an HFTextVectorizer and passing show_progress_bar=False, an error is thrown by SentenceTransformers — I haven't found another way of disabling the progress bar from polluting logs.

Expected behavior:
• The progress bar should be hidden when show_progress_bar=False.

Actual behavior:
• An error is thrown:

  File "/Users/raphaeldelio/Documents/GitHub/redis/devrel-content-crawler/.venv/lib/python3.11/site-packages/redisvl/utils/vectorize/text/huggingface.py", line 103, in _initialize_client
    self._client = SentenceTransformer(model, **kwargs)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Code that threw the error:

    def _get_router(self):
        """Get the singleton router instance"""
        if self._router is None:
            routes = self._create_routes()
            self._router = SemanticRouter(
                name="filter-router",
                vectorizer=HFTextVectorizer(
                    model="sentence-transformers/all-MiniLM-L6-v2",
                    **show_progress_bar=False**
                ),
                routes=routes,
                redis_url="redis://localhost:6379",
                overwrite=True
            )
        return self._router

[rbs333]

@raphaeldelio thanks for posting!

One thing to note is show_progress_bar is a option for the embed function. This will eliminate the the second tqdm progress bar in the example below (the first one will still appear for loading the model) but something to note.

Working on something where this can be set at the root level but figured I'd give that update in the short term

[raphaeldelio]

@rbs333 Is the model only loaded once? The cluttering I'm facing is with the multiple bars that show up when matching semantic routes or running the optimizer (especially)

[rbs333]

@raphaeldelio correct model is only loaded once. Adding a pr to silence this by default when invoked so you would see less.