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

Align factor coder into new framework #47

Merged
merged 8 commits into from
Jul 5, 2024

Conversation

peteryang1
Copy link
Collaborator

@peteryang1 peteryang1 commented Jul 5, 2024

PR包含了以下改动:

  1. 把factorimplementation更名为factorcoder避免factorimplementationevaluator这种confuse的变量名
  2. modelimplementation同样也进行更名
  3. 给taskgenerator和evaluator加上scenario的初始化参数,保证taskgenerator和evaluator与场景无关
  4. 把factor coder里面所有关于场景的prompt删除,改为从Scenario传入,方便factor implementation用于别的场景

📚 Documentation preview 📚: https://RDAgent--47.org.readthedocs.build/en/47/

@peteryang1 peteryang1 merged commit f15a5c2 into main Jul 5, 2024
4 of 8 checks passed
@you-n-g you-n-g deleted the xuyang1/align_factor_coder_into_new_framework branch August 2, 2024 08:31
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.

2 participants