feat: support optional query content #1097
Merged
Conversation
This file contains 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
iamjoel
changed the title
crazywoola
previously approved these changes
Sep 5, 2023
GarfieldDai
force-pushed
the
feat/generation-support-optional-query-content2
branch
from
d95d428 to
2ae1e58
Compare
GarfieldDai
force-pushed
the
feat/generation-support-optional-query-content2
branch
from
2ae1e58 to
737bf61
Compare
takatost
approved these changes
Sep 9, 2023
HuberyHuV1
pushed a commit
to HuberyHuV1/dify
that referenced
this pull request
Jul 22, 2024
Co-authored-by: Garfield Dai <dai.hai@foxmail.com>
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.
Query contentin the text generation application have been removed. Users can freely add any variables in the prompt, instead of appending the content of the query content at the end.queryparameter in thePOST /v1/completion-messagesendpoint of the App OpenAPI is expected to be completely removed in 3 months. Currently, it's optional and the parameter in API documentation has already been removed.To ensure that the previous applications can still function properly after removing the fixed parameters in the query content of the text generation application, we need to execute a command to fix the data:
Migration Guide
Enter the Docker Container: Before running the migration script, you need to enter the Docker container where your application is running. Use the following command:
docker exec -it docker-api-1 bashRun the Migration Script: Once inside the container, execute the following migration command. The migration script can be executed multiple times without side effects:
# --batch-size: Specifies the number of records migrated per batch, default is 500 flask update_app_model_configs --batch-size=500