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

[demos][python] Add a new Python PaLM demo: Docs Agent #116

Merged
merged 10 commits into from
Sep 18, 2023

Conversation

kyolee415
Copy link
Collaborator

Doc Agents allows PaLM API users to launch a chat application using their own documents as a dataset.

Doc Agents allows PaLM API users to launch a chat application using
their own documents as a dataset.
@kyolee415 kyolee415 requested a review from a team as a code owner September 1, 2023 19:51
@github-actions github-actions bot added the status:awaiting review PR awaiting review from a maintainer label Sep 1, 2023
Copy link
Member

@markmcd markmcd left a comment

Choose a reason for hiding this comment

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

Left a few basic comments. I didn't go over anything in detail, just spot checked a couple things and mainly checked the licenses.

Can you please include ownership changes in /CODEOWNERS and set up your unit test instrumentation too - can be another PR if you like, but please do it ASAP (e.g. no CODEOWNERS means we have to manually triage any PRs / issues that are filed).

demos/palm/python/docs-agent/CONTRIBUTING.md Outdated Show resolved Hide resolved
demos/palm/python/docs-agent/CONTRIBUTING.md Outdated Show resolved Hide resolved
demos/palm/python/docs-agent/README.md Show resolved Hide resolved
demos/palm/python/docs-agent/README.md Outdated Show resolved Hide resolved
demos/palm/python/docs-agent/pylintrc Show resolved Hide resolved
kyolee415 and others added 2 commits September 15, 2023 13:02
- Replace the `CONTRIBUTING.md` with Google's official one.
- Updated `REDAME.md` to adjust a link, edit a section, and include
  contributors.
- Run `black` on the `populate_vector_database.py` script.
- Update `CODEOWNERS`
CODEOWNERS Show resolved Hide resolved
Copy link
Member

@markmcd markmcd left a comment

Choose a reason for hiding this comment

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

Thanks!

CODEOWNERS Show resolved Hide resolved
demos/palm/python/docs-agent/pylintrc Show resolved Hide resolved
@kyolee415 kyolee415 merged commit 94c0ba0 into google:main Sep 18, 2023
2 checks passed
@github-actions github-actions bot removed the status:awaiting review PR awaiting review from a maintainer label Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:demos Update demos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants