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

genai: update pyproject as vertexai #420

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nobu007
Copy link
Contributor

@nobu007 nobu007 commented Aug 3, 2024

This PR aligns the pyproject.toml files for genai and vertexai, reducing maintenance costs.

Changes

  • packages is almost based by vertexai(especially types-protobuf helpful).
  • version is latest of genai and vertexai.
  • fix mypy ignore.
  • minar fix about SecretStr of test.

@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch from 3b64b20 to 3fbcd87 Compare August 18, 2024 04:10
@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch from 3fbcd87 to 5a5f8c1 Compare September 1, 2024 09:37
@nobu007 nobu007 marked this pull request as ready for review September 1, 2024 09:38
@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch 6 times, most recently from 9bf4893 to 35bb8f6 Compare September 1, 2024 13:19
@@ -46,6 +54,10 @@ codespell = "^2.2.0"
optional = true

[tool.poetry.group.test_integration.dependencies]
numexpr = { version = "^2.8.8", python = ">=3.9,<4.0" }
google-api-python-client = "^2.114.0"
google-cloud-datastore = "^2.19.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don't use this library in this project

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lkuligin
I updated PR. Remove unused libs.

numexpr = { version = "^2.8.8", python = ">=3.9,<4.0" }
google-api-python-client = "^2.114.0"
google-cloud-datastore = "^2.19.0"
langchain = { git = "https://github.com/langchain-ai/langchain.git", subdirectory = "libs/langchain" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here, why do we need this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lkuligin
I updated PR. Remove unused libs.

@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch from 35bb8f6 to 32655bc Compare October 12, 2024 08:26
@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch 10 times, most recently from 5779ee2 to 566a9ba Compare November 19, 2024 17:49
@nobu007 nobu007 force-pushed the genai_update_pyproject_as_vertexai branch from 566a9ba to 2ef7214 Compare November 19, 2024 17:58
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.

2 participants