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

Disable failed C++ Test #6823

Merged
merged 1 commit into from
Mar 26, 2024
Merged

Disable failed C++ Test #6823

merged 1 commit into from
Mar 26, 2024

Conversation

lsy323
Copy link
Collaborator

@lsy323 lsy323 commented Mar 26, 2024

Disable the cpp tests for now due to upstream change.

[ RUN      ] AtenXlaTensorTest.TestPinverse
unknown file: Failure
C++ exception with description "svd: LAPACK library not found in compilation

[ RUN      ] AtenXlaTensorTest.TestRandpermCPUFallback
unknown file: Failure
C++ exception with description "Need to provide pin_memory allocator to use pin memory.
Exception raised from GetCPUAllocatorMaybePinned at /home/lsiyuan/work/pytorch/aten/src/ATen/EmptyTensor.cpp:23 (most recent call first):

[ RUN      ] AtenXlaTensorTest.TestInverse
unknown file: Failure
C++ exception with description "Calling torch.linalg.lu_factor on a CPU tensor requires compiling PyTorch with LAPACK. Please use PyTorch built with LAPACK support.
Exception raised from apply_lu_factor at /home/lsiyuan/work/pytorch/aten/src/ATen/nati

@JackCaoG JackCaoG merged commit 2b66e93 into master Mar 26, 2024
2 checks passed
@JackCaoG JackCaoG deleted the lsiyuan/disable-cpp-tests branch March 26, 2024 20:19
@lsy323
Copy link
Collaborator Author

lsy323 commented Mar 26, 2024

Hi @changm, AtenXlaTensorTest.TestRandpermCPUFallback failed due to change in upstream and it's disabled for now. I found the test is added in #6482, would you please take a look when you get a chance? Thanks!

@lsy323
Copy link
Collaborator Author

lsy323 commented Mar 26, 2024

The failure due to the LAPACK dependency is tracked in #6824

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