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

SeqFilter output might cause warning messages and further correction steps than required #121

Open
greatfireball opened this issue Feb 7, 2018 · 0 comments

Comments

@greatfireball
Copy link
Member

I tried to correct a single sequence using proovread. During the estimation steps for masked positions I received the following errors/warnings:

Use of uninitialized value $bpN in division (/) at ../../proovread/bin/proovread line 1716, <$xgr> line 1.

Use of uninitialized value $bpN in division (/) at ../../proovread/bin/proovread line 1718, <$xgr> line 1.

Obviously, this resulted in

Masked : 0.0%
Masked < 92% total / > 3% new - continuing

Therefore, no correction step will be skipped even if everything was already masked.

Further investigation of that problem point to the SeqFilter script, which

"[..]might be ommited if to few sequences are retrieved[...]"

In my case, the N90 value was omitted and therefore, the value for masked positions is undefined.

I will provide a fix for that issue

greatfireball added a commit that referenced this issue Feb 7, 2018
Fixes #121

The number of fields in SeqFilter output might differ, the program now extracts the second and the last field. Therefore, it is guaranteed to work with a variable field number.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant