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

Integrate CodeIntepreter #736

Merged
merged 477 commits into from
Feb 5, 2024
Merged

Integrate CodeIntepreter #736

merged 477 commits into from
Feb 5, 2024

Conversation

garylin2099
Copy link
Collaborator

@garylin2099 garylin2099 commented Jan 12, 2024

Features

  • Planning ability for general Role through _plan_and_act function
  • A CodeInterpreter(Role) that could solve problems through coding. This role inherits the planning ability and can also use tools.
  • A MLEngineer(Role) that tailored for machine learning and data analysis
  • Both roles above can interact with human for plan confirmation or modification
  • A Planner and a data structure Plan were the support for role's planning capability.
  • Multiple Actions reusable across roles, incluing WriteCodeByGenerate, WriteCodeWithTools, ExecuteCode, etc. These enable a real-time code generation and display.

Feature Docs
TODO

Result
TODO


This change is Reviewable

orange-crow and others added 30 commits December 19, 2023 16:50
更新:修改execute_code 初始化,增加resume功能

See merge request agents/data_agents_opt!27
Add ablation experiments

See merge request agents/data_agents_opt!28
minor update: move action, fix circular import, add entry parameters

See merge request agents/data_agents_opt!29
Copy link
Owner

@geekan geekan left a comment

Choose a reason for hiding this comment

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

  1. Don't write comments that don't make sense, if the subclass's methods are nothing special, it should probably use the parent's comments
  2. Notice that some logic has actually changed boundaries in the modification and need to provide a test result

metagpt/roles/ml_engineer.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/libs/feature_engineering.py Outdated Show resolved Hide resolved
metagpt/tools/tool_data_type.py Outdated Show resolved Hide resolved
metagpt/tools/tool_types.py Outdated Show resolved Hide resolved
@garylin2099
Copy link
Collaborator Author

garylin2099 commented Feb 4, 2024

  1. Don't write comments that don't make sense, if the subclass's methods are nothing special, it should probably use the parent's comments
  2. Notice that some logic has actually changed boundaries in the modification and need to provide a test result
  1. Done
  2. https://github.com/geekan/MetaGPT/actions/runs/7775123241/job/21200724452?pr=736 All tests passed with a coverage no lower than dev. (The 7 failures are the same with current dev and are not related to this PR)

Copy link
Owner

@geekan geekan left a comment

Choose a reason for hiding this comment

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

LGTM

@geekan geekan merged commit 23c2762 into geekan:dev Feb 5, 2024
1 of 3 checks passed
@garylin2099 garylin2099 deleted the code_intepreter branch February 5, 2024 13:56
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.

7 participants