update protobuf API calls so it does not have to be pinned to protobuf-4

Open pavoljuhas opened this issue 1 year ago • 1 comments

Description of the issue

https://github.com/protocolbuffers/protobuf/commit/26995798757fbfef5cf6648610848e389db1fecf removed the including_default_value_fields argument which is used in cirq-google. This is temporarily fixed by pinning protobuf-4.x in #6650

TODO: update API calls to use the new argument always_print_fields_with_no_presence.

Cirq version

1.5.0.dev at 0aae3c122ad457cec884e8bfbe001afde9d17f5b.

Jun 25 '24 01:06 pavoljuhas

Let's update the relevant calls (or recompile the protos if necessary) to use the new call. Investigate compatibility with internal systems.

Can either @verult @senecameeks take this?

Jul 03 '24 17:07 dstrain115