Skip to content
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.

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

Protobuf can go back to protobuf>=3.12.2 #638

Closed
wants to merge 8 commits into from

Conversation

hayotensor
Copy link

Make sure to delete everything but: requirements.txt, there are other updates to RoutingTable.py to allow for updating timestamps to node_ids in the routing table for applications like Petals to use... Or even just make another commit

With protobuf>=3.12.2,<5.28.0 I receive the error:

google.protobuf.runtime_version.VersionError: Detected incompatible Protobuf Gencode/Runtime versions when loading runtime.proto: gencode 5.28.1 runtime 5.27.5. Runtime version cannot be older than the linked gencode version. See Protobuf version guarantees at https://protobuf.dev/support/cross-version-runtime-guarantee.

This should happen if you --force a pip install again and try to start a server in Petals.

This was fixed here: protocolbuffers/protobuf#18096 (It looks like)

Switching back to protobuf>=3.12.2 resolves this issue on Linux and WSL for me

@hayotensor hayotensor closed this Nov 21, 2024
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.

1 participant