-
Notifications
You must be signed in to change notification settings - Fork 865
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
Loosening protobuf version limit breaking downstream packages #716
Comments
It broke our build too. |
Hi @psfoley, does v2.6.2.2 work for you? |
Thanks @lanpa for working to address this! Manually installing v2.6.2.2 does fix the problem, but for prior releases of our package we had only required |
Hi @psfoley, I have yanked v2.6.2.1. Now |
Thanks @lanpa. We have protobuf 3.19 as a requirement in our framework, so now v2.6.2 is installed instead of v2.6.2.1 (because 2.6.2 is where the protobuf requirement was loosened). If v2.6.2 is yanked as well, everything should work again. Really appreciate all your help with this! |
I have just yanked v2.6.2! Sorry for the inconvenience. Please try again. |
That fixed it! Thanks so much @lanpa! |
Describe the bug
The recent removal of the protobuf version version limit (#712) has breaking implications for downstream packages dependent on tensorboardX and protobuf<3.20.
Minimal runnable code to reproduce the behavior
The text was updated successfully, but these errors were encountered: