Feat: get_label endpoint #144
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…sing the right template, revert this commit before merging the new template with main
…v/chatsky-ui into feat/context_endpoints
** Like this, the path will be relative to the working-dir instead of repo root
…es, also changed the database_path config to the unified context_storage path
…li.py updated for new config file settings
…ly opening it once instead of once per query (fix)
…oing it like in chatsky. Attempting to only connect once, testing for now
…hatsky does it. get_chat_ids() has to be done without that via sqlite3 library, though
… it to Context Storage
…age outside a Chatsky Pipeline. This may be a workaround, I will check with Chatsky devs
…d logging for exceptions and renamed the new endpoint according to review changes
…chatsky-ui into feat/labels_endpoint
…rations, if I'm not wrong
…, the tests just had some very specific problems
…ime, because of pytest causing an asyncio error (Extractor can only support a single Event loop, just like Chatsky databases)
…ery time because of pytest creating an asyncio Error, it's explained in label_endpoints PR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Added the
get_labelnew endpoint, takesrun_id,user_idandmessage_idas its arguments. Returns the node label of the current turn.Checklist
To Consider
.ignorefiles, scripts (such aslint), distribution manifest (if files are added/deleted)