OrensteinLab / SWOffinder

MIT License
1 stars 0 forks source link

Inclusion of a PAM-absence warning #6

Open MatthewPace98 opened 3 weeks ago

MatthewPace98 commented 3 weeks ago

Thank you for this fantastic tool. Just a minor suggestion: if the PAM is not detected in the list of sgRNAs, throw a warning.

I spent some time trying to figure out why the csv output did not contain the off-target PAM region when <Allow PAM edits> was TRUE. I now see that it was because my input sequences did not contain a PAM, and that this is written in the arguments specifications. It would have saved some stress if this was highlighted a little better, or if a warning was implemented :)