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

raise error when msg is invalid; fix docstr; improve ResponsiveAgent; update doc and packaging; capture ipython output; configurable default reply #1154

Merged
merged 63 commits into from
Aug 1, 2023

Conversation

sonichi
Copy link
Contributor

@sonichi sonichi commented Jul 29, 2023

Why are these changes needed?

  1. raise a ValueError when received msg is not a valid oai message in ResponsiveAgent.
  2. fix docstr.
  3. allow None to return in genreate_reply().
  4. add update_system_message().
  5. update doc.
  6. add color in print.
  7. improve packaging.
  8. capture ipython cell output in reply when executing in ipython.
  9. find code blocks with llm when regex fails - removed as it doesn't work reliably.
  10. make default_auto_reply configurable.

Related issue number

Checks

@sonichi sonichi temporarily deployed to openai July 31, 2023 18:40 — with GitHub Actions Inactive
@sonichi sonichi temporarily deployed to openai July 31, 2023 18:40 — with GitHub Actions Inactive
@sonichi sonichi temporarily deployed to openai July 31, 2023 18:40 — with GitHub Actions Inactive
@sonichi sonichi added this pull request to the merge queue Jul 31, 2023
@sonichi sonichi removed this pull request from the merge queue due to a manual request Jul 31, 2023
@sonichi sonichi temporarily deployed to openai July 31, 2023 22:38 — with GitHub Actions Inactive
@sonichi sonichi temporarily deployed to openai July 31, 2023 22:38 — with GitHub Actions Inactive
@sonichi sonichi temporarily deployed to openai July 31, 2023 22:38 — with GitHub Actions Inactive
@sonichi sonichi added this pull request to the merge queue Jul 31, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 1, 2023
@sonichi sonichi added this pull request to the merge queue Aug 1, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Aug 1, 2023
@sonichi sonichi added this pull request to the merge queue Aug 1, 2023
@sonichi sonichi changed the title raise error when msg is invalid; fix docstr; improve ResponsiveAgent; update doc and packaging; capture ipython output; find code blocks with llm when regex fails. raise error when msg is invalid; fix docstr; improve ResponsiveAgent; update doc and packaging; capture ipython output; configurable default reply Aug 1, 2023
Merged via the queue into main with commit c48babd Aug 1, 2023
16 checks passed
@sonichi sonichi deleted the validmsg branch August 1, 2023 04:11
@sonichi sonichi linked an issue Aug 1, 2023 that may be closed by this pull request
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.

termcolor is probably useful for autogen too
4 participants