Account created on 31 August 2008, over 16 years ago
#

Merge Requests

More

Recent comments

πŸ‡§πŸ‡ͺBelgium mpp

I'm OK with this, as ddev is the de facto standard for local Drupal development. Thanks!

πŸ‡§πŸ‡ͺBelgium mpp

Hi ipwa, thank you for reaching out. You've been added as co-maintainer!

πŸ‡§πŸ‡ͺBelgium mpp

"I also wound up thinking maybe you don't need that level of splitting, and just stored one vector."

While having a single embedding for one document has some benefits (e.g. cheaper, faster), it comes with some challenges:
- document size may be too large to create a single embedding, hence chunking is needed
- you could loose meaning/semantics

Depending on the context & the type of content, you may want to have a different embedding strategy. For instance paragraphs or sentence embeddings may be useful.

πŸ‡§πŸ‡ͺBelgium mpp

Same issue with aws.elastic-cloud.com

Any update on this?

πŸ‡§πŸ‡ͺBelgium mpp

mpp β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium mpp

The prompt should be translatable in order to support multilingual.

πŸ‡§πŸ‡ͺBelgium mpp

Thank you @larowlan. Shouldn't this be mentioned on the project page?

πŸ‡§πŸ‡ͺBelgium mpp

@bala_28 this patch fixes the following issue: "Structural error at security should be array".

The op mentioned a lot more issues.

πŸ‡§πŸ‡ͺBelgium mpp

Closing this after having talked to BradJones.

The latest dev version depends on this change to core which is not yet committed.

https://www.drupal.org/project/drupal/issues/3031367 ✨ Introduce "schematic" normalizers Active

πŸ‡§πŸ‡ͺBelgium mpp

Looking at the rout Pretty sure this qualifies as a config link:

πŸ‡§πŸ‡ͺBelgium mpp

Looking at the rout Pretty sure this qualifies as a config link:

πŸ‡§πŸ‡ͺBelgium mpp

mpp β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium mpp

Merge request should be against 4.x, not 3.x

πŸ‡§πŸ‡ͺBelgium mpp

mpp β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium mpp

Either the plugin.manager.vector_client or the openai.client that won't serialize properly.

Disabled caching as a workaround.

πŸ‡§πŸ‡ͺBelgium mpp

"This change will require a test case also to show the issue."

The failure actually shows that there is a bug as the current test doesn't provide a user with the 'access taxonomy overview' permission.

πŸ‡§πŸ‡ͺBelgium mpp

Added check on 'access taxonomy overview' permission.

πŸ‡§πŸ‡ͺBelgium mpp

This introduced an issue that in some cases the link is shown for users that don't have access to the vocabulary overview πŸ› Only show link to taxonomy overview when user has access Fixed .

πŸ‡§πŸ‡ͺBelgium mpp

Any update on this? Seems like the "official" module won't be upgraded either.

Production build 0.71.5 2024