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
[JIT] Imbue stringbuf with C locale #79929
Conversation
🔗 Helpful links
❌ 5 New Failures, 2 Base FailuresAs of commit 05f28af (more details on the Dr. CI page): Expand to see more
🕵️ 5 new failures recognized by patternsThe following CI failures do not appear to be due to upstream breakageslinux-binary-manywheel / manywheel-py3_10-cpu-test (1/5)Step: "Checkout PyTorch" (full log | diagnosis details | 🔁 rerun)
|
To prevent 12345 become "12,345" if locale is not "C", as shown in the following example: ```cpp int main() { std::locale::global(std::locale("en_US.utf-8")); std::stringstream ss; ss << "12345 in " << std::locale().name() << " locale is " << 12345 ; ss.imbue(std::locale("C")); ss << " but in C locale is " << 12345; std::cout << ss.str() << std::endl; } ```
17db387
to
05f28af
Compare
pip installing https://gha-artifacts.s3.amazonaws.com/pytorch/pytorch/2533367977/manywheel-py3_7-cuda11_6/torch-1.13.0.dev20220621%2Bcu116-cp37-cp37m-linux_x86_64.whl shows that problem is indeed gone |
@pytorchbot merge |
@pytorchbot successfully started a merge job. Check the current status here |
Merge failed due to Refusing to merge as mandatory check(s) pull failed for rule superuser |
@pytorchbot merge -f |
@pytorchbot successfully started a merge job. Check the current status here |
Hey @malfet. |
To prevent 12345 become "12,345" if locale is not "C", as shown in the following example: ```cpp int main() { std::locale::global(std::locale("en_US.utf-8")); std::stringstream ss; ss << "12345 in " << std::locale().name() << " locale is " << 12345 ; ss.imbue(std::locale("C")); ss << " but in C locale is " << 12345; std::cout << ss.str() << std::endl; } ``` Fixes pytorch#79583 Pull Request resolved: pytorch#79929 Approved by: https://github.com/davidberard98
Summary: To prevent 12345 become "12,345" if locale is not "C", as shown in the following example: ```cpp int main() { std::locale::global(std::locale("en_US.utf-8")); std::stringstream ss; ss << "12345 in " << std::locale().name() << " locale is " << 12345 ; ss.imbue(std::locale("C")); ss << " but in C locale is " << 12345; std::cout << ss.str() << std::endl; } ``` Fixes #79583 Pull Request resolved: #79929 Approved by: https://github.com/davidberard98 Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/4b6ba340e211503b758ea6c3c173e1e7bbd9c11c Reviewed By: atalman Differential Revision: D37316875 Pulled By: malfet fbshipit-source-id: d3900f532930850bd4973274c3274cf31d8b8d09
To prevent 12345 become "12,345" if locale is not "C", as shown in the following example: ```cpp int main() { std::locale::global(std::locale("en_US.utf-8")); std::stringstream ss; ss << "12345 in " << std::locale().name() << " locale is " << 12345 ; ss.imbue(std::locale("C")); ss << " but in C locale is " << 12345; std::cout << ss.str() << std::endl; } ``` Fixes #79583 Pull Request resolved: #79929 Approved by: https://github.com/davidberard98 Co-authored-by: Nikita Shulga <nshulga@fb.com>
To prevent 12345 become "12,345" if locale is not "C", as shown in the following example: ```cpp int main() { std::locale::global(std::locale("en_US.utf-8")); std::stringstream ss; ss << "12345 in " << std::locale().name() << " locale is " << 12345 ; ss.imbue(std::locale("C")); ss << " but in C locale is " << 12345; std::cout << ss.str() << std::endl; } ``` Fixes pytorch#79583 Pull Request resolved: pytorch#79929 Approved by: https://github.com/davidberard98
To prevent 12345 become "12,345" if locale is not "C", as shown in the
following example:
Fixes #79583