Skip to content

Fixes that needs to be available in the main branch in order to cherry-pick into Release branch - #49

Merged
vkarampudi merged 19 commits into
google:masterfrom
vkarampudi:testing3
Feb 5, 2026
Merged

Fixes that needs to be available in the main branch in order to cherry-pick into Release branch#49
vkarampudi merged 19 commits into
google:masterfrom
vkarampudi:testing3

Conversation

@vkarampudi

Copy link
Copy Markdown
Collaborator

No description provided.

vkarampudiand others added 19 commits April 3, 2025 22:22
Update 0.48.0 in version.py and RELEASE.md
…tion
Protobuf 4.x removed py_proto_library and changed proto_library behavior. This commit:
- Creates _py_proto_library_rule: a custom Bazel rule that accepts ProtoInfo or PyInfo,
runs protoc to generate Python _pb2.py files, and provides PyInfo for Python deps
- Replaces deprecated py_proto_library calls with custom rule implementation
- Configures protoc proto_path to include workspace dirs and external dependencies
- Adds local any.proto copies to bypass sandbox limitations in proto imports
- Applies compatibility patches to TensorFlow and TensorFlow Metadata dependencies
- Updates all s2t_proto_library_py calls to work with new implementation
Fixes: Protobuf 4.23.4 compatibility for struct2tensor build system
This patch extends tensorflow.patch with changes to support:
1. Abseil backward compatibility:
- Add absl_base_prefetch.h compatibility shim for older Abseil
versions that lack absl/base/prefetch.h
- Update prefetch includes across TensorFlow/TSL to use the shim
- Add inline implementation of NullTerminatedMessage for older
Abseil versions without StatusMessageAsCStr
2. Protobuf 4.x support:
- Refactor cc_proto_library to use native proto_library and
cc_proto_library rules instead of custom proto_gen
- Implement custom _tsl_py_proto_library_rule to replace the
built-in py_proto_library removed in Protobuf 4.x
- Update proto library generation to depend on generated
cc_proto_library targets
3. Build cleanup:
- Remove unused absl/strings:string_view dependency
- Update BUILD files to reflect new dependency structure
These changes enable building struct2tensor with older Abseil versions
while supporting Protobuf 4.x, improving compatibility across different
dependency versions.
Remove vendored google/protobuf/any.proto and obsolete third_party/README.md
Upgrade `struct2tensor` to Protobuf 4.25.6 with compatibility patches
Prefix cc_binary and cc_library with native. in struct2tensor.bzl
Use native prefix for Bazel rules in macros
Apply rules_cc patch before adding struct2tensor local repository to ensure proper Bazel configuration.
Fix struct2tensor integration in TensorFlow Serving Docker build
@vkarampudi
vkarampudi merged commit ce726b0 into google:masterFeb 5, 2026
2 checks passed
vkarampudi added a commit that referenced this pull request Feb 11, 2026
* Struct2Tensor 0.48.0 Release
PiperOrigin-RevId: 743712449
* Build documentation with `mkdocs` (#38)
* Add minimum viable mkdocs configuration
* Add examples section
* Add links to api docs and overviews
* Get api docs up and running
* Add `__all__` to `__init__.py` files to include correct classes
This is just a first step to including the correct items in the api docs
* Exclude test modules from docs
* Use correct module
* Add docs deployment workflow
* Include all correct modules/classes/functions
* Fix button rendering
* Fix formatting, including admonitions
* Fix broken link
* Use README.md as homepage for docs
* Fix links
* Remove debug triggers from docs deployment workflow
* Remove deprecated api docs
* Fix triggers
* Move docs requirements to common location
* Only trigger on pushes to master
* Remove unnecessary newline characters
* Move docs location to default `docs` directory
* only internal visibility change, no external change
PiperOrigin-RevId: 755419474
* Automated Code Change
PiperOrigin-RevId: 796979132
* Automated Code Change
PiperOrigin-RevId: 803375108
* no-op
PiperOrigin-RevId: 809038483
* Added pytype None checks to calculate.py
PiperOrigin-RevId: 845226580
* Replace unicode escaped characters in ipynb files
PiperOrigin-RevId: 856192266
* Fixes that needs to be available in the main branch in order to cherry-pick into Release branch (#49)
* Update WORKSPACE
* Update workspace.bzl
* Migrate to Protobuf 4.23.4: custom Bazel rule for Python proto generation
Protobuf 4.x removed py_proto_library and changed proto_library behavior. This commit:
- Creates _py_proto_library_rule: a custom Bazel rule that accepts ProtoInfo or PyInfo,
runs protoc to generate Python _pb2.py files, and provides PyInfo for Python deps
- Replaces deprecated py_proto_library calls with custom rule implementation
- Configures protoc proto_path to include workspace dirs and external dependencies
- Adds local any.proto copies to bypass sandbox limitations in proto imports
- Applies compatibility patches to TensorFlow and TensorFlow Metadata dependencies
- Updates all s2t_proto_library_py calls to work with new implementation
Fixes: Protobuf 4.23.4 compatibility for struct2tensor build system
* Add compatibility patches for older Abseil and Protobuf 4.x
This patch extends tensorflow.patch with changes to support:
1. Abseil backward compatibility:
- Add absl_base_prefetch.h compatibility shim for older Abseil
versions that lack absl/base/prefetch.h
- Update prefetch includes across TensorFlow/TSL to use the shim
- Add inline implementation of NullTerminatedMessage for older
Abseil versions without StatusMessageAsCStr
2. Protobuf 4.x support:
- Refactor cc_proto_library to use native proto_library and
cc_proto_library rules instead of custom proto_gen
- Implement custom _tsl_py_proto_library_rule to replace the
built-in py_proto_library removed in Protobuf 4.x
- Update proto library generation to depend on generated
cc_proto_library targets
3. Build cleanup:
- Remove unused absl/strings:string_view dependency
- Update BUILD files to reflect new dependency structure
These changes enable building struct2tensor with older Abseil versions
while supporting Protobuf 4.x, improving compatibility across different
dependency versions.
* chore: upgrade Protobuf dependency from 4.23.4 to 4.25.6
* Move TFMD patch to tfmd.patch; Protobuf 4.x compatibility
Remove vendored google/protobuf/any.proto and obsolete third_party/README.md
* fix: Use native prefix for Bazel rules in macros
Prefix cc_binary and cc_library with native. in struct2tensor.bzl
* Update Dockerfile
* Fix struct2tensor integration in TensorFlow Serving Docker build
Apply rules_cc patch before adding struct2tensor local repository to ensure proper Bazel configuration.
* Update RELEASE.md to sync with main branch
* Update version.py
---------
Co-authored-by: Gagandeep Singh <gdp.1807@gmail.com>
* docs(release): Update RELEASE.md and version for 0.48.0
* Update Dockerfile to build with serving version 2.17.1
* Build and stage dynamic op libraries before wheel creation (#51)
Add explicit build and copy steps for dynamic op libraries (.so files) in
build_common.sh. This ensures all six dynamic library targets are built and
copied to struct2tensor/ops/ before the wheel is packaged.
Previously, the stamp_wheel step in build_manylinux.sh would fail with
patchelf errors ("No such file or directory") because the .so files were
expected in struct2tensor/ops/ but were not being copied from bazel-bin.
This fix:
- Explicitly builds each dynamic library target
- Copies the built .so files from bazel-bin to the staging directory
- Runs before bazel run :build_pip_package to ensure files are present
Fixes the missing _decode_proto_map_op.so, _decode_proto_sparse_op.so and
related dynamic library warnings during wheel packaging.
* Enable OSS Bazel builds by fixing dependencies and proto paths (#50)
- Add py_proto_library macro for OSS compatibility
- Replace gunit_main with googletest gtest_main
- Comment out Google-internal dependencies and missing test files
- Fix proto import to use workspace-relative path
- Remove invalid third_party load statements
Fixes build errors when running `bazel build //...` in OSS environment.
Co-authored-by: Madhur Karampudi <142544288+vkarampudi@users.noreply.github.com>
---------
Co-authored-by: struct2tensor-team <tensorflow-extended-nonhuman@googlegroups.com>
Co-authored-by: Gagandeep Singh <gdp.1807@gmail.com>
Sign up for freeto 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.

3 participants

@vkarampudi@genehwung@czgdp1807