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

rebuild BaseResearchAgent with unified function logic #138

Merged
merged 7 commits into from
May 30, 2024

Conversation

lwaekfjlk
Copy link
Member

@lwaekfjlk lwaekfjlk commented May 29, 2024

Closes #42

πŸ“‘ Description

rebuild read_paper, think_idea, write_paper, write_paper_review, write_paper_meta_review, write_rebuttal, discuss

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • Branch name follows type/descript (e.g. feature/add-llm-agents)
  • Ready for code review

β„Ή Additional Information

@lwaekfjlk
Copy link
Member Author

rebuild read_paper, think_idea, write_paper, write_paper_review, write_paper_meta_review, write_rebuttal, discuss

@lwaekfjlk lwaekfjlk requested a review from Monstertail May 29, 2024 22:10
@lwaekfjlk
Copy link
Member Author

@Monstertail @timsanders256 Please also help check the reorged codebase. I believe it is much cleaner now.

@lwaekfjlk lwaekfjlk requested review from chengzr01 and removed request for Monstertail May 29, 2024 22:10
@lwaekfjlk
Copy link
Member Author

@ft2023 check this for the progress.

@lwaekfjlk
Copy link
Member Author

so currently, each funciton call in agent_base takes one Profile Class type as input and take some research progress as output.

@lwaekfjlk lwaekfjlk changed the title rebuild agent-base with unified api calling rebuild agent-base with unified api calling logic May 29, 2024
@lwaekfjlk lwaekfjlk changed the title rebuild agent-base with unified api calling logic rebuild agent-base with unified function logic May 29, 2024
@lwaekfjlk lwaekfjlk changed the title rebuild agent-base with unified function logic rebuild BaseResearchAgent with unified function logic May 30, 2024
@lwaekfjlk
Copy link
Member Author

I replaced the research trend into research insight.

@lwaekfjlk
Copy link
Member Author

I confirmed all the tests pass.

@chengzr01
Copy link
Member

I think the code looks great and ready for merge! Thanks for your efforts!

@lwaekfjlk lwaekfjlk merged commit 754606f into main May 30, 2024
0 of 12 checks passed
@lwaekfjlk lwaekfjlk deleted the feature/add-serializer-into-agent-base branch September 8, 2024 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants