search

mdshw5 / pyfaidx

Efficient pythonic random access to fasta subsequences
https://pypi.python.org/pypi/pyfaidx
Other
459 stars 75 forks source link

Update __init__.py #123

Closed KwatMDPhD closed 7 years ago

mdshw5 commented 7 years ago

So the BGZF support works well enough for you that you want the warning removed?

KwatMDPhD commented 7 years ago

BGZF worked for me and the linked URL broke. So I wanted to remove that warning message.

From the design perspective, I think either: 1) BGZF allowed (giving that it works) and warning removed, or 2) BGZF not allowed is better. Having something somehow working is dangerous.

What do you think?

mdshw5 commented 7 years ago

I agree with you. It looks like I'm raising errors for any method calls that can not work using BGZF files, so this warning is no longer needed. Thanks!