Skip to content

WIP prompt reactions#388

Draft
mgood wants to merge 1 commit intomainfrom
mg-branch-4
Draft

WIP prompt reactions#388
mgood wants to merge 1 commit intomainfrom
mg-branch-4

Conversation

@mgood
Copy link
Copy Markdown
Collaborator

@mgood mgood commented May 7, 2025

documenting current status of the prompt reaction

This is based on a reaction I wrote earlier for the bridge assistant responses. Running the LLM command directly works, but I’m getting a 400 error now from the llama-3 service when I try to run it via quickjs.

documenting current status of the prompt reaction
@netlify
Copy link
Copy Markdown

netlify Bot commented May 7, 2025

Deploy Preview for projectsubstrate canceled.

Name Link
🔨 Latest commit 087708b
🔍 Latest deploy log https://app.netlify.com/sites/projectsubstrate/deploys/681beaa2d529f100083cc8bb

@ajbouh
Copy link
Copy Markdown
Owner

ajbouh commented May 8, 2025

The general problem here is that the golang msg client isn't properly setting the model parameter to "/res/model/huggingface/local". This happens properly in the javascript msg client. I am still working on a fix. One possible (temporary) workaround might be to set this parameter directly. I am still trying to figure out why it isn't being properly set. It seems to be an issue with how we merge values when we use the SetPath logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants