-
Notifications
You must be signed in to change notification settings - Fork 43
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
Permissions error on /tmp/ir_dataset directory due to multiple users on the same server #206
Comments
I ran into same issue, but not quite sure if it is the same bug as yours @mitgosp Tried running:
After download finished (45 mins), got this error:
P/s: Full cell output:
|
@mitgosp Sorry if this is a stupid question, but how do I utilize the |
Hi @yuenherny -- it looks like this is a different issue. Do you have multiple processes open using ir_datasets? (E.g., multiple notebook instances)? As files are downloading, only a single process can access them on Windows. |
Hi @seanmacavaney , thanks for the prompt response. Nope, I guess the process is open because I tried to download multiple parts of the dataset - Now that I managed to download (after restarting my laptop), I get another error:
Possibly due to incomplete download? |
Gotcha -- thanks! This is a bug, as it should close the file in this case so others can use it. I'll look into fixing this.
Yep, something went wrong with the download. It's not safe to use this version because the contents could be different, or you may be missing some records. |
Describe the bug
When more than one users on the same server or device use the ir_datasets to fetch documents, then the permission denied error might be encountered if one of the users does not have write access to the already created directory
Affected dataset(s)
This issue does not affect datasets
To Reproduce
Steps to reproduce the behavior:
PermissionError: [Errno 13] Permission denied: '/tmp/ir_datasets/tmp3sn3tbic'
Expected behavior
When multiple users are using the package on the same device, some additional checks would need to be in place to avoid permission errors. For example, the ir_directory directory that is created for tmp files could be prefaced by a username to avoid such conflicts.
Additional context
This issue can be bypassed by utilizing the IR_DATASETS_TMP environment variable.
The text was updated successfully, but these errors were encountered: