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

add $genome permission check #125

Merged
merged 1 commit into from
Sep 30, 2022
Merged

add $genome permission check #125

merged 1 commit into from
Sep 30, 2022

Conversation

with9
Copy link
Contributor

@with9 with9 commented May 3, 2022

Thank you for your work.
LAI did not run correctly.While running I get the following error $genome.tbl: No such file or directory
After checking I found out that it is due to the permissions of the $genome file.After runing RepeatMasker -e ncbi -pa $threads -q -no_is -norna -nolow -div 40 -lib $genome.LTRlib.fa -cutoff 225 $genome i got this error.

analyzing file $genome
FastaDB::_cleanIndexAndCompact() - Error could not open file ../LTR/RM_166328.TueMay32136282022/$genome: Permission denied
 at /miniconda3/envs/lai/bin/RepeatMasker line 739.

and after chmod +w $genome , everything runs correctly

I add two line to check if $genome is writable.

@oushujun
Copy link
Owner

@with9 Thank you for your contribution. RepeatMasker or LTR_retriever will not write the genome file, so they do not require the genome to be writeable. You may need to make the genome readible. Can you check if making/checking the genome readible is enough?

@with9
Copy link
Contributor Author

with9 commented May 11, 2022

@with9 Thank you for your contribution. RepeatMasker or LTR_retriever will not write the genome file, so they do not require the genome to be writeable. You may need to make the genome readible. Can you check if making/checking the genome readible is enough?

i think RepeatMasker will first copy the genome.fa ,during copying the permissions not modified

with@fedora /d/w/E/l/r/p/r/a/test> ll
total 9.9G
-r--r--r--. 1 with with 9.8G May 11 06:38 Es.bp.p_ctg.fa
-r--r--r--. 1 with with 115M May 11 06:38 Es.bp.p_ctg.fa.LTRlib.fa
with@fedora /d/w/E/l/r/p/r/a/test> RepeatMasker -e ncbi -pa 2 -q -no_is -norna -nolow -div 40 -lib Es.bp.p_ctg.fa.LTRlib.fa -cutoff 225 Es.bp.p_ctg.fa
RepeatMasker version 4.1.2-p1
Search Engine: NCBI/RMBLAST [ 2.11.0+ ]
Using Custom Repeat Library: Es.bp.p_ctg.fa.LTRlib.fa

Building general libraries in: /home/with/miniconda3/envs/lai/share/RepeatMasker/Libraries//general

analyzing file Es.bp.p_ctg.fa
FastaDB::_cleanIndexAndCompact() - Error could not open file /data2/with_data2/Eb_project/lai/ret_retriver/parallel/repaeatannotion/annotate_out/test/RM_1576551.WedMay110639422022/Es.bp.p_ctg.fa: Permission denied
 at /home/with/miniconda3/envs/lai/bin/RepeatMasker line 739.


with@fedora /d/w/E/l/r/p/r/a/test> ll RM_1576551.WedMay110639422022/Es.bp.p_ctg.fa                                                   
-r--r--r--. 1 with with 9.8G May 11 06:39 RM_1576551.WedMay110639422022/Es.bp.p_ctg.fa

@with9 with9 closed this by deleting the head repository Sep 30, 2022
@oushujun oushujun reopened this Sep 30, 2022
@oushujun
Copy link
Owner

Thank you for your recommendatian. In general, it would require the user to make sure things are writable. But adding a check will definitely help to diagnose what may be the cause of errors.

@oushujun oushujun merged commit 8d95fce into oushujun:master Sep 30, 2022
@GallVp
Copy link

GallVp commented Jul 18, 2024

This issue was resolved by RepeatMasker on Nov 2, 2022. Please see,

rmhubley/RepeatMasker@3d83626

https://github.com/rmhubley/RepeatMasker/blob/3d83626710649d98222da7e0940a756138878fc0/RepeatMasker#L741-L743

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.

None yet

3 participants