Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

small changes to verbose message format #150

Merged
merged 2 commits into from
Sep 1, 2023

Conversation

johnnykfeng
Copy link
Contributor

Quick fix for issue #149

@amirfz
Copy link
Contributor

amirfz commented Sep 1, 2023

@johnnykfeng while you're at it, could you make a few other small changes?

1- "I for the inconvenience, but it seems that you have exceeded ..." fix the typo here -> "Sorry for the inconvenience, but it seems that you have exceeded ..."

2- display a message instead of the json object:

current:

🔊Command: 
 {'name': 'Search', 'args': {'query': 'open source frameworks and SDKs for building applications with large language models'}}

desired:

🛠 SEARCH \n
❓ query: open source frameworks and SDKs for building applications with large language models

Copy link
Contributor

@amirfz amirfz left a comment

Choose a reason for hiding this comment

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

@johnnykfeng
Copy link
Contributor Author

johnnykfeng commented Sep 1, 2023

@johnnykfeng while you're at it, could you make a few other small changes?

1- "I for the inconvenience, but it seems that you have exceeded ..." fix the typo here -> "Sorry for the inconvenience, but it seems that you have exceeded ..."

2- display a message instead of the json object:

current:

🔊Command: 
 {'name': 'Search', 'args': {'query': 'open source frameworks and SDKs for building applications with large language models'}}

desired:

🛠 SEARCH \n
❓ query: open source frameworks and SDKs for building applications with large language models

Done! Check out second commit in this PR

As for the typo, idk where that is in the code since I didn't write it

@amirfz amirfz merged commit 94947e7 into Aggregate-Intellect:main Sep 1, 2023
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