Account created on 30 October 2008, about 16 years ago
#

Merge Requests

More

Recent comments

🇩🇪Germany marcus_johansson

Thank you jurgenhaas and jan kellerman - after some discussions on Slack and in a thread here 🌱 Solution for checking localStorage Active , all is looking good for me. The descriptive texts are a little bit short, but I saw on Slack that this is not to think about yet.

We will release a new version of the AI module that will take care of the inherent issues outside of this issue soon.

🇩🇪Germany marcus_johansson

Thank you @jan kellerman - the #1 was the piece of information I was missing. Then your MR makes total sense.

Regarding your MR - we moved the project into the core AI module, so I have replicated the issue here, if you want to push the MR again to get proper attribution: https://www.drupal.org/project/ai/issues/3494806 📌 Dont set localStorage by default Active . Sorry about the inconvinience and confusion. I will credit you regardless. Let me know.

I'll set this to fixed, thank you for the thorough answer!

🇩🇪Germany marcus_johansson

marcus_johansson made their first commit to this issue’s fork.

🇩🇪Germany marcus_johansson

We switch over to Verdana, sans-serif for now on all themes instead. Anyone interested in having more consistent fonts will have to enable it on their site themselves.

🇩🇪Germany marcus_johansson

Because of this issue, I think this is a frontend loading issue. https://www.drupal.org/project/ai/issues/3494808 🐛 Strange icon in Deepchat window Active

@wouters_f - could you check your console log?

🇩🇪Germany marcus_johansson

@wouters_f - Its the copy button, I'm wondering if something goes wrong when everything loads for you. Which OS/Browser are you using?

🇩🇪Germany marcus_johansson

Paul, could you repush your changes, I don't see a way directly in Gitlab to revert it to that change. And then I'll merge.

🇩🇪Germany marcus_johansson

Thank you everyone, this is getting merged into dev and the next release.

🇩🇪Germany marcus_johansson

There are a trigger rollback that the actions can use if they want now.

🇩🇪Germany marcus_johansson

I escalated the issue itself, because its a obvious flaw if it does this. I tested with beta5 and could not replicate the issue. Could you try to update to beta5 and see if you still experience the issue?

🇩🇪Germany marcus_johansson

marcus_johansson changed the visibility of the branch 3494285-translating-unpublished-content to hidden.

🇩🇪Germany marcus_johansson

Looks good, getting merged. Thank you Andrew.

🇩🇪Germany marcus_johansson

They were required earlier as well, we just missed to add it.

We will abstract away the functionality of it at an later stage (scraping / document reading) - see here for example: #3482004

I realize because of your comment that the experimental lifecycle should be set there - this module is experimental for now. #3493842

The thing you can do when you have unstructured installed is that you can for instance upload a Word file or a Excel file with some data and it can create a node type and fields based on this, or with simple crawler you can point to a website you own and create fields based on the markup/content.

But we will make these optional, so you still can use a prompt with a normal LLM or an image if you have a vision model.

I created an issue for it here: #3493847

🇩🇪Germany marcus_johansson

This issue is very old, but I thought I would revive it :)

I saw this LinkedIn post https://www.linkedin.com/posts/chris-kelly-b22425148_drupal-openai-activ... and did try his images on slightly larger resolution and it got the text out.

More importalty - I was looking at pricing on OpenAI and noticed that there is not difference between 500x500 and 200x200 on gpt4-o. The max price would be $0.000638 (excluding input and output text), if we scale the largest of width and height to 500px. See https://openai.com/api/pricing/

Since most people use OpenAI still, it would make sense to go up to this resolution in size, since in many cases it won't affect the price. WDYT @jurriaanroelofs?

🇩🇪Germany marcus_johansson

There is a issue for it here: https://www.drupal.org/project/ai_provider_openai/issues/3489954 🐛 Unnecessary validation error Active . Its been assigned to someone for sometime, but if they don't fix it we will soon.

Closing this, since it doesn't belong here.

🇩🇪Germany marcus_johansson

Hi gto1 - you could try to get the dev release. I just merged it.

🇩🇪Germany marcus_johansson

It didn't connect label and readable_name. Have changed that to label.

🇩🇪Germany marcus_johansson

Reviewed and its passing the testing. Thanks Vivek!

🇩🇪Germany marcus_johansson

This means that the request is taking longer than 30 seconds that you have as your max request timeout. You can raise it here: https://www.jeffgeerling.com/blog/2016/increase-guzzle-http-client-reque...

There is an pull request to add this per provider, but currently you have to use the above solution.

🇩🇪Germany marcus_johansson

I think we can set this to fixed now?

🇩🇪Germany marcus_johansson

Its for sure external moderation, we should add some validation and description there. OpenAI uses the moderation endpoint by default.

The external moderation is just for adding one providers moderation endpoint over another provider, so if you add OpenAI over OpenAI it might cause circular dependency.

But this is a good bug catch and should be fixed - I created an issue for it here: https://www.drupal.org/project/ai/issues/3493419 🐛 Do not allow for the same providers moderation to be used in external moderation Active

Production build 0.71.5 2024