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.

Sign up for GitHub

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

Jump to bottom

Rebuild AgentScope Studio #238

Merged
merged 92 commits into from
Jun 9, 2024
Merged

Rebuild AgentScope Studio #238

merged 92 commits into from
Jun 9, 2024

Conversation

pan-x-c
Copy link
Collaborator

@pan-x-c pan-x-c commented May 20, 2024 •
edited by DavdGao
Loading

Description

  1. refactor AgentScope Studio and WebUI
  2. merge AgentScope Workstation into AgentScope Studio
  3. add distributed agent server manager into AgentScope Studio

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has passed all tests
  • Docstrings have been added/updated in Google Style
  • Documentation has been updated
  • Code is ready for review

pan-x-c added 21 commits May 10, 2024 13:14
@pan-x-c
Split the rpc_agent module into multiple submodules
51ab7a8
@pan-x-c
rename RpcAgentServerLauncher to AgentServerLauncher
dbf5b18
@pan-x-c
add as_server into setup and support graceful shutdown
7997f7a
@pan-x-c
add server_id
a92eac2
@pan-x-c
update tutorial for agent server
9163241
@pan-x-c
update tutorial for agent server
31228b6
@pan-x-c
update tutorial for agent server
23a8f5f
@pan-x-c
fix windows add_signal_handler
918c750
@pan-x-c
rename agentserverlauncher
8f3eaa2
@pan-x-c
add model_config_path and update tutorial
49c5dcf
@pan-x-c
update tutorial
b635e35
@pan-x-c
merge conflict
939615f
@pan-x-c
fix pre-commit
c96156d
@pan-x-c
fix comments
bf86cc5
@pan-x-c
fix comments
c9eaf2d
@pan-x-c
Merge branch 'main' into feature/pxc/app_server_id
6ba2d82
@pan-x-c
add websocket for message handling
0c1fbc1
@pan-x-c
init agentscope studio plus
7215974
@pan-x-c
Merge branch 'main' into feature/pxc/refactor_studio
8ac0595
@pan-x-c
add api for get messages and runs
e06a0ec
@pan-x-c
update api of get messages and runs
271c8e4
DavdGao
DavdGao reviewed May 21, 2024
View reviewed changes
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

Plz see inline comments, and suggested to use Pydantic to format to protocol

src/agentscope/web/_app.py Outdated Show resolved Hide resolved
@pan-x-c pan-x-c changed the title [WIP] Rebuild AgentScope Studio Rebuild AgentScope Studio Jun 7, 2024
pan-x-c and others added 15 commits June 7, 2024 14:07
@pan-x-c
set default sorter for api invocation
ad80d2c
@DavdGao
Fix bug in missing value warning; Add default handler for roles;
3514c9a
@pan-x-c
remove katex
aa134ce
@pan-x-c
fix details btn
7559d62
@DavdGao
Rename functions into private ones
3916aa7
@pan-x-c
fix url type
39f2195
@pan-x-c
Merge pull request #5 from DavdGao/feature/pxc/refactor_studio_dawei
3d2b184 
Convert into private functions; Fix bug in workstation
@DavdGao
[WIP]
2c7a89f
@DavdGao
Remove language switch functionality; Press send button to enable mes…
7fdb552 
…sage sending; Generate random color and icon according to the runtime id;
@DavdGao
Fix the ImportError for studio client
e95550f
@DavdGao
Move studio from agentscope/web/studio into agentscope/studio
8220f73
@DavdGao
Change the entry point of AgentScope Studio
f254138
@DavdGao
Fix error in import _studio_client; Clean unnecessary logging in js a…
e34bfc4 
…nd flask server
@DavdGao
Remove unnecessary change; fix bug in conversation example.
9833c26
@DavdGao
Modify the cache dir
65c729e
pan-x-c
pan-x-c commented Jun 9, 2024
View reviewed changes
src/agentscope/studio/_client.py Outdated Show resolved Hide resolved
src/agentscope/studio/_client.py Outdated Show resolved Hide resolved
src/agentscope/studio/_app.py Outdated Show resolved Hide resolved
DavdGao
DavdGao approved these changes Jun 9, 2024
View reviewed changes
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

LGTM

@DavdGao DavdGao merged commit 3560d8e into modelscope:main Jun 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rayrayraykk rayrayraykk rayrayraykk left review comments

@qbc2016 qbc2016 qbc2016 left review comments

@DavdGao DavdGao DavdGao approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pan-x-c @qbc2016 @rayrayraykk @DavdGao