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

Consistent type hints for Sequence in interfaces #4783

Open
ekzhu opened this issue Dec 21, 2024 · 4 comments
Open

Consistent type hints for Sequence in interfaces #4783

ekzhu opened this issue Dec 21, 2024 · 4 comments

Comments

@ekzhu
Copy link
Collaborator

ekzhu commented Dec 21, 2024

Currently List and Sequences are mixed in agentchat.base types. Make them all Sequence as it is more general.

  • ChatAgent.produced_message_types
  • Response.inner_messages
  • TaskRunner.run
  • TaskRunner.run_stream
@ekzhu ekzhu added this to the 0.4.0 milestone Dec 21, 2024
@kartikx
Copy link
Contributor

kartikx commented Dec 21, 2024

Hi, if this is a good first issue can I pick this up? I would like to contribute to AutoGen.

@ekzhu
Copy link
Collaborator Author

ekzhu commented Dec 22, 2024

Sure. There is a time urgency though if you are comfortable with. Let us know asap if you are blocked or otherwise unable to continue. We would like to get this done in a few days. Please read CONTRIBUTING.md to get familiar with the development process.

@kartikx
Copy link
Contributor

kartikx commented Dec 22, 2024

Thanks for the info @ekzhu. I might not be able to get it done within the deadline so please don't be blocked on me.

@ekzhu
Copy link
Collaborator Author

ekzhu commented Dec 22, 2024

Thanks for the info @ekzhu. I might not be able to get it done within the deadline so please don't be blocked on me.

No problem. There are many issues tagged with 0.4.1 milestones and beyond. You can take a look at those and see if you have any suggestions.

The best kind of contributions we can get right now are bug reports and you actively trying out the API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants