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

[bug] Fix run_tests.py --with-offline-cache #7507

Merged

Conversation

PGZXB
Copy link
Contributor

@PGZXB PGZXB commented Mar 7, 2023

Issue: #7002

Brief Summary

Fixed bug reported by https://github.com/taichi-dev/taichi/actions/runs/4346468687/jobs/7592576659 :

Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "tests/run_tests.py", line 233, in print_and_remove
    os.listdir(os.path.join(tmp_cache_file_path, subdir)))
NotADirectoryError: [Errno 20] Not a directory: '/var/folders/1r/fk_r0s1d4ss8m19rg11bq1j80000gp/T/tmpcnyh7mz1/Tbdbfefe5b5fe6db167e250d15bb1c4cc797b2beee4fa7a09be9a6cff8c060101-metal.tic'

Quick repro on master:

python run_tests.py -v -t4 -a vulkan,cpu abs --with-offline-cache

...
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "..\taichi\tests\run_tests.py", line 233, in print_and_remove
    os.listdir(os.path.join(tmp_cache_file_path, subdir)))
NotADirectoryError: [WinError 267] The directory name is invalid. : 'C:\\Users\\xxx\\AppData\\Local\\Temp\\tmpcs7z9cpq\\T1571a996babd2f9b7a24eeba9d8450fd767cf939b8d717941f629550302d6408-vulkan.tic'

On this branch (fixed the bug):

python run_tests.py -v -t4 -a vulkan,cpu abs --with-offline-cache

...
Summary of testing the offline cache:
    Simple statistics: {'llvm': 19, '*.tic': 18}
    Size of cache files: 646.04 KB

@netlify
Copy link

netlify bot commented Mar 7, 2023

Deploy Preview for docsite-preview canceled.

Name Link
🔨 Latest commit 90213f9
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/64071b3b336aae0009834143

@PGZXB PGZXB requested a review from ailzhang March 7, 2023 14:26
Copy link
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ailzhang ailzhang merged commit 9afa90a into taichi-dev:master Mar 8, 2023
quadpixels pushed a commit to quadpixels/taichi that referenced this pull request May 13, 2023
Issue: taichi-dev#7002 

### Brief Summary
#### Fixed bug reported by
https://github.com/taichi-dev/taichi/actions/runs/4346468687/jobs/7592576659
:
```
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "tests/run_tests.py", line 233, in print_and_remove
    os.listdir(os.path.join(tmp_cache_file_path, subdir)))
NotADirectoryError: [Errno 20] Not a directory: '/var/folders/1r/fk_r0s1d4ss8m19rg11bq1j80000gp/T/tmpcnyh7mz1/Tbdbfefe5b5fe6db167e250d15bb1c4cc797b2beee4fa7a09be9a6cff8c060101-metal.tic'
```
#### Quick repro on master:
`python run_tests.py -v -t4 -a vulkan,cpu abs --with-offline-cache`
```
...
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "..\taichi\tests\run_tests.py", line 233, in print_and_remove
    os.listdir(os.path.join(tmp_cache_file_path, subdir)))
NotADirectoryError: [WinError 267] The directory name is invalid. : 'C:\\Users\\xxx\\AppData\\Local\\Temp\\tmpcs7z9cpq\\T1571a996babd2f9b7a24eeba9d8450fd767cf939b8d717941f629550302d6408-vulkan.tic'
```
#### On this branch (fixed the bug):
`python run_tests.py -v -t4 -a vulkan,cpu abs --with-offline-cache`
```
...
Summary of testing the offline cache:
    Simple statistics: {'llvm': 19, '*.tic': 18}
    Size of cache files: 646.04 KB
```
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