Visitar URL original
fix: Import grpc only for type checking in errors.py by tokoko · Pull Request #4533 · feast-dev/feast · GitHub
Skip to content

fix: Import grpc only for type checking in errors.py #4533

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.

Sign up for GitHub

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

Merged
tokoko merged 15 commits into from
Sep 22, 2024
Merged

fix: Import grpc only for type checking in errors.py #4533

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
b8bc6d7
fix: grpc import error
tokoko Sep 18, 2024
7dd19bb
fix: loosen protobuf build requirement
tokoko Sep 18, 2024
ce3b89f
fix: pin grpcio-tools version
tokoko Sep 18, 2024
4cef278
fix: revert build-system in pyproject
tokoko Sep 18, 2024
551e108
fix: add manual install of setuptools and grpcio-tools
tokoko Sep 18, 2024
0d5c139
fix: remove incorrect pixi call
tokoko Sep 18, 2024
e0ffa9a
fix: add in-function imports in errors.py
tokoko Sep 18, 2024
8eec3e7
fix: formatting
tokoko Sep 18, 2024
cbe330c
Merge branch 'master' into test-import-no-extras
tokoko Sep 18, 2024
7094342
fix: merge changes from master
tokoko Sep 19, 2024
bc5e86b
merge from master
tokoko Sep 19, 2024
a53001f
fix: add line endings
tokoko Sep 19, 2024
51584b6
fix: add line endings
tokoko Sep 19, 2024
c4d54f7
Merge remote-tracking branch 'origin' into test-import-no-extras
tokoko Sep 22, 2024
a375685
chore: remove proto generation from make commands
tokoko Sep 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: add manual install of setuptools and grpcio-tools 
Signed-off-by: tokoko <togurgenidze@gmail.com>
  • Loading branch information
@tokoko
tokoko committed Sep 18, 2024
commit 551e10812b7a13b1c9fbfa487d2b9545d7f31d75
7 changes: 2 additions & 5 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,18 +37,15 @@ build: protos build-java build-docker

# Python SDK

install-python-dependencies:
python -m piptools sync sdk/python/requirements/py$(PYTHON_VERSION)-requirements.txt
pip install --no-deps .
python setup.py build_python_protos --inplace

install-python-dependencies-uv:
uv pip sync --system sdk/python/requirements/py$(PYTHON_VERSION)-requirements.txt
uv pip install --system setuptools grpcio-tools==1.56.2
uv pip install --system --no-deps .
python setup.py build_python_protos --inplace

install-python-dependencies-uv-venv:
uv pip sync sdk/python/requirements/py$(PYTHON_VERSION)-requirements.txt
uv pip install setuptools grpcio-tools==1.56.2
uv pip install --no-deps .
python setup.py build_python_protos --inplace

Expand Down
14 changes: 0 additions & 14 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,17 +1,3 @@
[build-system]
requires = [
"grpcio-tools==1.56.2",
"grpcio>=1.56.2,<2",
"mypy-protobuf==3.1",
"protobuf>=4.24.0,<5.0.0",
"pybindgen==0.22.0",
"setuptools>=60",
"setuptools_scm>=6.2",
"sphinx!=4.0.0",
"wheel",
]
build-backend = "setuptools.build_meta"

[tool.setuptools_scm]
# Including this section is comparable to supplying use_scm_version=True in setup.py.

Expand Down