Conversation
Added a `PROMPTY_CHAT_PARSER_DISABLE_IMAGE_PARSING=true` environment variable to disable image parsing in the chat parser. In my use case, I am using Prompty to send a prompt to the LLM that contains Markdown that should be converted to a different format. I do not need Prompty to parse the images and try to send them to the LLM. In fact, this is inconvenient because I don't want to have to copy the images just so Prompty can read them. An environment variable seems kind of janky perhaps, but it's the best solution I could come up with that doesn't require a lot of changes, because there's a fairly deep call stack so in order to pass a parameter, it would require modifying a lot of code. I'm open to other ideas though.
sethjuarez
left a comment
There was a problem hiding this comment.
This is a good idea - let me think about how to do this in a more declarative way.
|
Bump |
|
Could this please be reviewed? This is actually causing a problem for me right now. |
|
I have made some changes, but I haven't put in a PR yet though, as the newer version of Prompty is quite different, and seemingly inconsistent. For example, this is in the And if I try to use this example, I get an exception as it doesn't expect Also when |
|
OK fixed a few things PR #274. @sethjuarez - could you please review? |
Added a
PROMPTY_CHAT_PARSER_DISABLE_IMAGE_PARSING=trueenvironment variable to disable image parsing in the chat parser.In my use case, I am using Prompty to send a prompt to the LLM that contains Markdown that should be converted to a different format. I do not need Prompty to parse the images and try to send them to the LLM. In fact, this is inconvenient because I don't want to have to copy the images just so Prompty can read them.
An environment variable seems kind of janky perhaps, but it's the best solution I could come up with that doesn't require a lot of changes, because there's a fairly deep call stack so in order to pass a parameter, it would require modifying a lot of code. I'm open to other ideas though.