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

Agents app #37

Merged
merged 16 commits into from
Apr 3, 2023
Merged

Agents app #37

merged 16 commits into from
Apr 3, 2023

Conversation

Obs01ete
Copy link
Collaborator

No description provided.

@Obs01ete Obs01ete changed the title Agents app Draft: Agents app Mar 29, 2023
@Obs01ete
Copy link
Collaborator Author

@lightaime review pls

@Obs01ete Obs01ete changed the title Draft: Agents app Agents app Apr 1, 2023
@Obs01ete
Copy link
Collaborator Author

Obs01ete commented Apr 1, 2023

@lightaime PR is ready, pls review and merge

Copy link
Member

@lightaime lightaime left a comment

Choose a reason for hiding this comment

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

Awesome! I like it. Leave some minor comments. Please merge it after you fix them. Also please remember to fix these flake8 issues: https://github.com/lightaime/camel/actions/runs/4585128885/jobs/8097075241?pr=37. Thanks!


try:
session = RolePlaying(assistant, user, original_task,
with_task_specify=True, with_task_planner=False)
Copy link
Member

Choose a reason for hiding this comment

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

It would be great if we can also expose with_task_specify to the UI. So people can choose if the task will be specified by CAMEL or not.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Will put it in the backlog

Copy link
Member

Choose a reason for hiding this comment

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

Sounds good.

return state, state.chat, gr.update(), gr.update()

try:
assistant_msg, user_msg = state.session.step(state.saved_assistant_msg)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
assistant_msg, user_msg = state.session.step(state.saved_assistant_msg)
assistant_msgs, user_msgs = state.session.step(state.saved_assistant_msg)

Comment on lines 237 to 238
u_msg = user_msg[0]
a_msg = assistant_msg[0]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
u_msg = user_msg[0]
a_msg = assistant_msg[0]
u_msg = user_msgs[0]
a_msg = assistant_msgs[0]

Comment on lines +308 to +325
assistant_ta = gr.TextArea(label="Assistant role (EDIT ME)",
lines=1, interactive=True)
with gr.Column(scale=1):
user_dd = gr.Dropdown(user_roles, label="Example user roles",
value=user_role, interactive=True)
user_ta = gr.TextArea(label="User role (EDIT ME)", lines=1,
interactive=True)
with gr.Column(scale=1):
gr.Markdown(
"## CAMEL: Communicative Agents for \"Mind\" Exploration"
" of Large Scale Language Model Society\n"
"Github repo: [https://github.com/lightaime/camel]"
"(https://github.com/lightaime/camel)")
with gr.Row():
with gr.Column(scale=9):
original_task_ta = gr.TextArea(
label="Give me a preliminary idea (EDIT ME)",
value=default_task, lines=1, interactive=True)
Copy link
Member

Choose a reason for hiding this comment

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

It would be great if we can add a markdown doc to explain the three EDIT ME fields.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Put where? To the GUI as a static text?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, we can do that. Giving more explanations would be helpful for users

@Obs01ete Obs01ete merged commit 961a0dc into master Apr 3, 2023
1 of 2 checks passed
@Obs01ete Obs01ete deleted the agents_app branch April 3, 2023 15:30
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.

None yet

2 participants