Philip Meier
Philip Meier
> However i took some time to find some official reference to back up what I say Sorry, I should have been more clear. A screenshot or even your testimony...
`/chats/{id}/answer` https://github.com/Quansight/ragna/blob/4a667f9aced1970f8eab8aa9e01aaf8c67b05db0/ragna/deploy/_api/core.py#L258-L264 returns a `Message` https://github.com/Quansight/ragna/blob/4a667f9aced1970f8eab8aa9e01aaf8c67b05db0/ragna/deploy/_api/schemas.py#L54-L61 So, directly, no document is returned here at all. The document is part of the `Source`s that are being returned https://github.com/Quansight/ragna/blob/4a667f9aced1970f8eab8aa9e01aaf8c67b05db0/ragna/deploy/_api/schemas.py#L35-L41 Finally we arrive...
I've played around with it for a bit and I think this is more a feature request than a bug. That being said, I think the change is for the...
I see that we are missing forward slashes for the `auth` and `logout` endpoints: https://github.com/Quansight/ragna/blob/38f75f6d88e74f72c030ef50045315b81eb203aa/ragna/deploy/_ui/app.py#L109-L115 @pierrotsmnrd Do you remember if this was intentional? I'm not saying that this is the...
@aktech confirmed that #276 indeed fixed the issue. However, we have for more instances of hardcoded paths for the login mechanism: - https://github.com/Quansight/ragna/blob/ea9ffeb526098c0d31c001bee2c69bbb84eb999d/ragna/deploy/_ui/app.py#L66 - https://github.com/Quansight/ragna/blob/ea9ffeb526098c0d31c001bee2c69bbb84eb999d/ragna/deploy/_ui/app.py#L76 - https://github.com/Quansight/ragna/blob/ea9ffeb526098c0d31c001bee2c69bbb84eb999d/ragna/deploy/_ui/app.py#L92 - https://github.com/Quansight/ragna/blob/ea9ffeb526098c0d31c001bee2c69bbb84eb999d/ragna/deploy/_ui/logout_page.py#L22 They...
@aktech This is solved with the PRs you have send, correct?
Shower thought: what if we create a new `Corpus` class that can be passed as `Rag().chat(documents=Corpus(), ...)`. Internally, if someone passes documents as it is done now, we just create...
@peachkeel Re delete: This was originally discussed in #62. We ended up merging #67, which added a delete endpoint to the API, which is likely what you are using in...
Sure, a PR works as well. I never meant for you to post all the code in the discussion itself, but rather create a discussion and link the code from...
I've spent a good deal of time thinking about this over the past few weeks and I think I found a proper solution that I'll propose here. This is going...