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

restriction enzymes overhang missing #3

Open
fgvieira opened this issue Oct 12, 2017 · 3 comments
Open

restriction enzymes overhang missing #3

fgvieira opened this issue Oct 12, 2017 · 3 comments

Comments

@fgvieira
Copy link

Dear all,

I've been looking using the restriction modules in Bioperl and I think that, for some enzymes, it is not properly calculating the overhang sequence. For example, with EcoRI it looks fine and gives AATT:

$ perl -e 'use Bio::Restriction::Enzyme; $re = Bio::Restriction::Enzyme->new(-name=>"EcoRI",-site=>"GAATTC", -cut=>1); print $re->overhang_seq()."\n"'
AATT

But, (e.g.) if you use TasI, Bioperl cannot identify any overhang:

$ perl -e 'use Bio::Restriction::Enzyme; $re = Bio::Restriction::Enzyme->new(-name=>"TasI",-site=>"AATT", -cut=>0); print $re->overhang_seq()."\n"'

As I see it, both should give the same overhang, no?
cheers,

@hyphaltip
Copy link
Member

There are a few places in the code where
if( $cut ) ... or if ( $self->{_cut} )
needs to be rewritten
if ( defined $cut ) {}
I have a fix I can patch in - not sure what branch we are working on any more though - help? @cjfields .

hyphaltip referenced this issue in bioperl/bioperl-live Oct 12, 2017
Fix bug #253 where need to use `defined` instead of just a true/false test since the value can be 0
@hyphaltip
Copy link
Member

See bioperl/bioperl-live#254 but we need to probably also push this to any release branches as this is only on master. Can a current release developer take a look?

@cjfields
Copy link
Member

@hyphaltip we're running off master for the latest releases. I'm in Africa but will be back this weekend, can take a look when I have a bit better internet (it's been reeaaaaaaally inconsistent).

@carandraug carandraug transferred this issue from bioperl/bioperl-live Dec 5, 2018
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

No branches or pull requests

3 participants