Increment requirement to Cirq to 0.13.1

[tonybruguier]

For issue #357 we would want to increase the version of Cirq used to be able to use new objects.

[MichaelBroughton]

It looks like cirq-rigetti is hurting us here. Can we see if just moving to depend on cirq-core and cirq-google fixes this issue ?

[tonybruguier]

@MichaelBroughton
Thanks! Excluding the Rigetti deps indeed helped, but then I encoutered another issue with the version of protobuf. I put more detail over at:
quantumlib/Cirq#4473

[MichaelBroughton]

Oops, looks like: quantumlib/Cirq#4167 might have broken us. We may have to take some extra steps here to be sure and "demote back down" any CCX or CCZ into ControlledGate(sub_gate=...) so that things keep working as intended.

[tonybruguier]

I updated the code to add the issue that we had discussed. @MichaelBroughton It's ready for review when you have the time.

[tonybruguier]

I took the liberty of bumping the requirements to v0.13.1

[MichaelBroughton]

This looks great! One question: Have we looked into this: https://googleapis.dev/python/protobuf/latest/google/protobuf/message.html#google.protobuf.message.Message.SerializeToString ?

Maybe instead of adding these two new functions program_from_tensor_to_ascii_proto we could just add an argument to convert_to_tensor like deterministic_proto_serialization and see if that fixes things ?

[tonybruguier]

It worked indeed. Thanks! I named the parameter deterministic_proto_serialize because the style guide limits things to 30 characters and chose its default value to be False to leave the serialization unchanged by default.

PTAL.

[MichaelBroughton]

LGTM