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

feat: Add GAIA benchmark #1181

Merged
merged 64 commits into from
Dec 8, 2024
Merged

feat: Add GAIA benchmark #1181

merged 64 commits into from
Dec 8, 2024

Conversation

liuxukun2000
Copy link
Collaborator

Description

task #640

Motivation and Context

Why is this change required? What problem does it solve?

add GAIA as a agent evaluation benchmark

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds core functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (update in the documentation)
  • Example (update in the folder of example)

Implemented Tasks

  • Subtask 1
  • Subtask 2
  • Subtask 3

Checklist

Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

  • I have read the CONTRIBUTION guide. (required)
  • My change requires a change to the documentation.
  • I have updated the tests accordingly. (required for a bug fix or a new feature)
  • I have updated the documentation accordingly.

camel/benchmarks/gaia.py Show resolved Hide resolved
camel/benchmarks/base.py Show resolved Hide resolved
examples/benchmarks/gaia.py Outdated Show resolved Hide resolved
results.jsonl Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/base.py Outdated Show resolved Hide resolved
camel/benchmarks/base.py Show resolved Hide resolved
camel/benchmarks/base.py Outdated Show resolved Hide resolved
camel/benchmarks/base.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
camel/benchmarks/gaia.py Outdated Show resolved Hide resolved
examples/benchmarks/gaia_results.jsonl Show resolved Hide resolved
Copy link
Member

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

Thanks @liuxukun2000 ! Left some comments and added commit here:d76f284
d8ce302
feel free check the change, I will merge the PR first, let me know if you have any further question



toolkit = CodeExecutionToolkit(verbose=True)
runtime = DockerRuntime("xukunliu/camel").add(
Copy link
Member

Choose a reason for hiding this comment

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

use RemoteHttpRuntime instead for easier set up

"""
return self.run_vector_retriever(query, contents, **kwargs) # type: ignore[arg-type]

def reset(self, **kwargs: Dict[str, Any]) -> bool:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
def reset(self, **kwargs: Dict[str, Any]) -> bool:
def reset(self, **kwargs: Any) -> bool:

bool: Whether the reset was successful.
"""
path = Path(self.vector_storage_local_path or os.getcwd())
task_id = str(kwargs.get("task_id", uuid.uuid1()))
Copy link
Member

Choose a reason for hiding this comment

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

better use uuid4

Comment on lines +161 to +165
def run( # type: ignore[override]
self,
agent: ChatAgent,
on: Literal["train", "valid", "test"],
level: Union[int, List[int], Literal["all"]],
Copy link
Member

Choose a reason for hiding this comment

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

this method could be refactored for better readability and maintainence

Copy link
Collaborator

@Zhangzeyu97 Zhangzeyu97 left a comment

Choose a reason for hiding this comment

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

Thanks @liuxukun2000 !

@Wendong-Fan Wendong-Fan merged commit c63f782 into master Dec 8, 2024
6 checks passed
@Wendong-Fan Wendong-Fan deleted the feat/gaia branch December 8, 2024 13:26
@willshang76 willshang76 mentioned this pull request Dec 9, 2024
13 tasks
@Wendong-Fan Wendong-Fan mentioned this pull request Dec 28, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

6 participants