Skip to content

Conversation

@lukasdotcom
Copy link
Member

This will be used by context_chat to allow user to download the logs. Note that streaming will not work correctly until this pr is released: nextcloud/app_api#608

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Copy link
Contributor

@kyteinsky kyteinsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@lukasdotcom
Copy link
Member Author

I changed the url here and in the pr for context_chat

@lukasdotcom lukasdotcom requested a review from kyteinsky July 9, 2025 16:27
Comment on lines 5 to 7
import zipfile

from starlette.responses import FileResponse
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah and this should be moved below too with the rest of the imports.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved and kept alphabetical order

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Copy link
Contributor

@kyteinsky kyteinsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@lukasdotcom lukasdotcom merged commit 9391bfe into nextcloud:master Jul 9, 2025
9 checks passed
@lukasdotcom lukasdotcom deleted the feat/log-endpoint branch July 9, 2025 16:58
@kyteinsky kyteinsky mentioned this pull request Jul 21, 2025
kyteinsky added a commit that referenced this pull request Jul 21, 2025
## 4.4.0 - 2025-07-21

### Fixed
- improve source tracking so no file stat is lost (#190) @kyteinsky
- improve OCS signing error messages (#189) @kyteinsky
- handle encrypted pdf decryption error (#195) @kyteinsky

### Changed
- maintenance update (#184) @kyteinsky
- update issue template to attach logs (#193) @lukasdotcom
- bump llama_cpp_python (#196) @kyteinsky

### Added
- add doc search endpoint (#185) @kyteinsky
- pre download the tokenizer instead of mid operation (#191) @kyteinsky
- add endpoint for downloading logs (#192) @lukasdotcom
- use supervisord to manage the processes (#194) @kyteinsky

Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants