Piero Savastano
Piero Savastano
`why` will be a plugin in `v2`, closing this for now as it was not even an issue (my bad)
added V2 tag as a todo list
Awesome thanks for the care in details. Will check ASAP
> is this PR still valid? Yes, needs a couple of tweaks but it is good
Sorry will not merge, only bug fixing for `v1`. In `v2` vector memory will be a plugin, if you are still interested we can move this PR there
Thanks never realized it was running twice XD
Merging soon
> Should i make these changes across all the files in core? Only endpoints for now! So `cat/routes` folder Assigning to you thanks 🙏
Sorry I closed because I did not understand the question Sounds right to give a warning and not a crash
In `v2` LLM execution is encapsulated in a try/except, so anything going wrong is passed as an error in the streaming response, failing gracefully. I consider this fixed, closing it...