Leonardo Pinheiro
Leonardo Pinheiro
> For `create_stream` I think a warning may not be sufficient, because the caller is not notified gracefully -- only the user inspecting the output is notified. It's gonna be...
> Hi, is this still being worked on? Getting killed by rate limits today and was hoping someone was working on a handler. Let me try to finish it up.
@ultmaster , may I help with this?
I started looking into this. At the moment there are some dependency conflicts that need to be resolved. The solutions would be to either constraint the python version as =...
Thanks @ekzhu and @rysweet . This should be ready for review now. Still needs improvements as mentioned in the description, but the tools can be used. I used the following...
@jackgerrits , I had to add `verride-dependencies` for `pydantic` and `tenacity` because the current version of pydantic is below their minimum requirement and there is a conflict with llamaindex which...
> Thank you! More documentation would help me review this PR. I would like to be able to build the docs page on this PR and see the example. @gagb...
> Let's add some unit tests? See the code coverage result. Is it possible to run a simple set up procedure with mini data set, perhaps generated? How much data...
@ekzhu @jackgerrits , I added the data in a conftest file. Since we are mocking the LLM calls it wont matter as much
@ekzhu , CI and docstrings fixed. I also update all examples to use component config but didn't create a copy in the graphrag sample to avoid a lot of duplicate...