-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AI Returns text as the language - Cell Won't Be Executable In RunMe #125
Comments
Can we enforce sometype of guardrail in the code? |
Actually I'm not sure where the bug is. Here's the logs from Foyle
We can clearly see the Agent.Generate has a block with the language set to
However when I look at the Raw markdown that gets saved in RunMe I see ```text {"id":"01HZ0GA3WXJD028PNFWAXFQR4V"} foyle train ``` So the language got changed somewhere.
|
I added some logging in commit 51341f4 Here are the responses
So it looks like language is set to bash
But in the markdown this ends up being ```text {"id":"01HZ0H5FVY07KGKVWKHQZG4ZTD"} # Execute the command to train Foyle train_foyle_command_here ``` |
…has no effect. It still gets converted to text in RunMe
I think the bug is here It doesn't look like we are setting languageId when inserting the cell. |
* See jlewi/foyle#125 the language id isn't being set on cells inserted from Foyle * As a result the language gets set to `text` and they aren't runnable.
* See jlewi/foyle#125 the language id isn't being set on cells inserted from Foyle * As a result the language gets set to `text` and they aren't runnable.
* See jlewi/foyle#125 the language id isn't being set on cells inserted from Foyle * As a result the language gets set to `text` and they aren't runnable.
I'm seeing examples where the AI returns a code cell with language
text
.RunMe won't know how to execute this.
This is related to #114. That issue surfaced an issue with nested blocks. It looks like there are also problems with the AI returning
text
. We should be able to verify this by looking at the Agent trace.In my case one of the generated block ids where I observed this was
01HYZWP2288PJY0CV53WAK6632
The text was updated successfully, but these errors were encountered: