Skip to content

Relax protobuf upper bound to <6.0.0 for lqp compatibility#158

Merged
gkastrinis merged 1 commit intomainfrom
relax-protobuf-upper-bound
Mar 20, 2026
Merged

Relax protobuf upper bound to <6.0.0 for lqp compatibility#158
gkastrinis merged 1 commit intomainfrom
relax-protobuf-upper-bound

Conversation

@gkastrinis
Copy link
Copy Markdown
Contributor

@gkastrinis gkastrinis commented Mar 20, 2026

Summary

  • Changes protobuf<5.0.0protobuf<6.0.0 in both setup.py and requirements.txt
  • Unblocks lqp>=0.5.0, which requires protobuf >=5.26 at runtime (uses runtime_version API)
  • Safe to do: the generated _pb2 files use the new-style BuildTopDescriptorsAndMessages API which is compatible with protobuf 5.x

Context

lqp==0.5.0 and rai-sdk==0.7.7 are currently incompatible because rai-sdk pins protobuf<5.0.0 while lqp needs >=5.26 at runtime. No lock file regeneration can resolve this — the fix must happen here.

🤖 Generated with Claude Code

lqp>=0.5.0 uses the protobuf runtime_version API which requires >=5.26.
The old <5.0.0 cap blocked lqp from being installed alongside rai-sdk.
The generated _pb2 files use the new-style BuildTopDescriptorsAndMessages
API which is compatible with protobuf 5.x, so no runtime breakage.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gkastrinis gkastrinis requested review from Akrion and meruyert93 March 20, 2026 09:39
@gkastrinis gkastrinis merged commit f9f711c into main Mar 20, 2026
@gkastrinis gkastrinis mentioned this pull request Mar 20, 2026
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