Hi! I am one of the maintainers, but have not been involved with IWC for a while. Original author of the module is @woprrr and @phenaproxima seems to be the most active lately. I would prefer if they would weigh in. With that said, I am OK with granting @mably committ access and I can do so if they don't respond.
I'd give them some more time. Please get back to me if they don't (email me as I am not monitoring this issue).
slashrsm → created an issue.
Great catch! My bad for only testing on llama. I assumed prefix refers to ollama app, not the model.
I implemented the new function for all providers in the module. For some I was not able to find an endpoint that would give me the size. For those I used a workaround where I generated an embedding for a pre-defined string and used the size of the returned array. In order to avoid repetitive calls I also cached the result.
dan2k3k4 → credited slashrsm → .
I fixed the linting errors.
I am not sure if we should allow 'not provided' as an answer. I think that we should require the vector size in all cases. Is there a valid case for not providing while supporting embeddings? The zero response was meant only as a temporary workaround until we implement it properly for all providers in this issue. If we agree that this is the approach to take I will attempt to do it.
After watching Driesnote and looking into AI module → a bit I realized that we are basically re-implementing their provider plugins here. In order to avoid that I decided to depend on the AI module for providers. Updated MR assumes/uses ✨ Provide embeddings vector size Active , which add vector size function that we rely on.
The merge request adds the new function to the interface as proposed. It also implements it for Ollama and OpenAI providers. I give a shot at implementing it for the other providers if the approach seems sensible.
slashrsm → created an issue.
slashrsm → changed the visibility of the branch query_side to hidden.
Still work in progress, but the idexing side of things work.
slashrsm → created an issue.
catch → credited slashrsm → .
slashrsm → created an issue.
I am open to adding DrDam as a co-maintainer. Can you work on a few issues in the Crop API issue queue and help move them into thh RTBC state. Get in touch with me (through d.o. contact form) after doing so and we'll continue from there.
nod_ → credited slashrsm → .
https://github.com/Lullabot/php-webdriver/pull/10/files was merged so we don't need to patch any more.
Nigel Cunningham → credited slashrsm → .
LGTM
Fix to a specific version of selenium addon until it gets fixed.
slashrsm → created an issue.
I can confirm that I can pass the headers into the OTEL client, which allows me to authorize with Grafana Cloud.
Added Grafana cloud docs link.
catch → credited slashrsm → .
In case anyone needs a fork with this patch on top of the current HEAD to point composer to: https://github.com/slashrsm/cookieconsent
+1 Works great, but we need a D10 release.
slashrsm → created an issue.
Attached patch fixed the problem.
slashrsm → created an issue.
This is still not a part of a relese and it took me a while to figure out the problem myself and realizing only after that that the issue was already fixed in -dev. Could we get a new release?