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

Turn agent_eval into a proper module to include it in Python package #3543

Closed
wants to merge 3 commits into from

Conversation

shtratos
Copy link
Member

@shtratos shtratos commented Sep 18, 2024

Why are these changes needed?

Without init,py it seems agent_eval directory is not included in pyautogen Python package.

So the only way to use agent_eval is via cloning the repo and adding it to the sys.path, which is awkward.

Related issue number

Closes #3497

Checks

Without __init__,py it seems `agent_eval` directory is not included in pyautogen Python package.

So the only way to use `agent_eval` is via cloning the repo and adding it to the `sys.path`, which is awkward.
Copy link
Member

@jackgerrits jackgerrits left a comment

Choose a reason for hiding this comment

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

Thanks!

@jackgerrits
Copy link
Member

@shtratos can you fix the code formatting error?

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.

[Issue]: Unable to import agenteval via pyautogen
2 participants