Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix CRISPR parser #302

Merged
merged 3 commits into from
Jul 16, 2024
Merged

Fix CRISPR parser #302

merged 3 commits into from
Jul 16, 2024

Conversation

oschwengers
Copy link
Owner

We wrongly assumed that a double digit number is correct for matching the CRISPR spacer length. However, there are shorter spacer lengths reported #299. Hence, we have to fix this by relaxing the regex to a single digit CRISPR spacer length.

The assumed minimal length of CRISPR spacers was reduced from at least 10 down to at least 1 in the regex.
@oschwengers oschwengers added the bug Something isn't working label Jul 16, 2024
@oschwengers oschwengers added this to the v1.9.4 milestone Jul 16, 2024
@oschwengers oschwengers self-assigned this Jul 16, 2024
@oschwengers oschwengers merged commit f90c836 into main Jul 16, 2024
12 checks passed
@oschwengers oschwengers deleted the fix-crispr-parser branch July 16, 2024 13:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant