fix for passing query parameter to procedure search #542
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.
Describe the changes you have made:
The query parameter was not passed properly to the open procedures search. It needs to be converted to a string to be properly URL encoded, and requests.get need to be called with params= (not json=)
I've debug and tested that it provide results that seem to make sense (eg. searching for email return related procedures. youtube seems to be returning relevant procedures). I'm unclear if passing the message in open_ai_messages is the best format to get results from this API end-point.
I wanted to keep the change to a minimum. But it might be nice to do better error handling here. It would be also really nice not to have to go to a remote server (but do the embedding search locally).
Reference any relevant issue (Fixes #000)
I have tested the code on the following OS:
AI Language Model (if applicable)