-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Develop reformating prompts and readme for offline learner #19
Conversation
vyokky
commented
Mar 9, 2024
- Reformating prompts on examples and API calls
- remove request header defined in the main function
- Add readme and examples for offline learner.
ACTION_SELECTION_PROMPT: "ufo/prompts/base/action_selection.yaml" # The prompt for the action selection | ||
ACTION_SELECTION_PROMPT: "ufo/prompts/base/action_selection_rag.yaml" # The prompt for the action selection | ||
|
||
APP_SELECTION_EXAMPLE_PROMPT: "ufo/prompts/examples/app_example.yaml" # The prompt for the app selection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May need to specify what's the difference between this prompt and previous app_selection prompt. Both called the prompt for the app selection
ufo/llm/llm_call.py
Outdated
@@ -25,6 +47,8 @@ def get_gptv_completion(messages, headers): | |||
return: The response of the request. | |||
""" | |||
aad = configs['API_TYPE'].lower() == 'azure_ad' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a checker variable.
aad = configs['API_TYPE'].lower() == 'azure_ad' | |
is_aad = configs['API_TYPE'].lower() == 'azure_ad' |
ufo/llm/prompt.py
Outdated
prompts += document | ||
prompts += "\n" | ||
prompts += "\n\n" | ||
return prompts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The type of the return is a prompt string. The name of prompts may cause confusing. Why it is prompts instead of prompt?
return: The prompt for retrieved documents. | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could specify return type, in function definition or comment line.
ufo/llm/prompt.py
Outdated
return prompts | ||
|
||
|
||
|
||
def api_prompt_helper(apis: dict, verbose: int = 1): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May specify return type.
def api_prompt_helper(apis: dict, verbose: int = 1): | |
def api_prompt_helper(apis: dict, verbose: int = 1) -> list[str]: |
@@ -33,6 +33,10 @@ Both agents leverage the multi-modal capabilities of GPT-Vision to comprehend th | |||
|
|||
|
|||
## 📢 News | |||
- 📅 2024-03-XX: New Release for v0.0.1! Check out our exciting new features: | |||
1. Our UFO framework now support RAG from offline document and online Bing search. | |||
2. We now support creating your help documents for each Windows app to become an app expert. Check XX for more details! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it a missing or TBD as expected?
ufo/ui_control/control.py
Outdated
""" | ||
Get the application name of the window. | ||
:param window: The window to get the application name. | ||
:return: The application name of the window. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:return: The application name of the window. | |
:return: The application name of the window. Empty string ("") if failed to get the name. |