Skip to content

Conversation

@DavdGao
Copy link
Member

@DavdGao DavdGao commented Mar 21, 2025

Description

In this PR, we will abandon some unused classes and pipelines from AgentScope to keep the code clean, and prepare for the next important update.

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

@DavdGao DavdGao changed the title [WIP] Remove Operator class and unused pipelines from agentscope Remove Operator class and unused pipelines from agentscope Mar 21, 2025
@DavdGao
Copy link
Member Author

DavdGao commented Mar 21, 2025

@rayrayraykk @qbc2016 Please take a look at this PR to ensure the workstation works normally.

Copy link
Member

@rayrayraykk rayrayraykk left a comment

Choose a reason for hiding this comment

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

Please change the implementation of Pipeline in compile function` to the new one.

@DavdGao DavdGao requested a review from ZiTao-Li March 24, 2025 03:21
@DavdGao
Copy link
Member Author

DavdGao commented Mar 24, 2025

Please change the implementation of Pipeline in compile function` to the new one.

Modified in the lastest commit

@rayrayraykk rayrayraykk requested a review from qbc2016 March 24, 2025 05:49
DavdGao added 4 commits March 25, 2025 17:22
# Conflicts:
#	examples/conversation_basic/conversation.py
#	src/agentscope/agents/__init__.py
#	src/agentscope/agents/_operator.py
#	src/agentscope/agents/agent.py
#	src/agentscope/pipelines/functional.py
#	src/agentscope/pipelines/pipeline.py
#	tests/rpc_agent_test.py
Copy link
Member

@qbc2016 qbc2016 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 31e73db into agentscope-ai:main Mar 25, 2025
13 checks passed
@DavdGao DavdGao deleted the dev/remove_pipeline branch May 29, 2025 13:32
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.

3 participants