Bug #21399 for Bio-Phylo: to_newick should not use caching

Fri Sep 08 20:12:30 2006 easmith [...] beatrice.rutgers.edu - Ticket created

Subject:	to_newick should not use caching
Date:	Fri, 8 Sep 2006 20:16:17 -0400
To:	bug-bio-phylo [...] rt.cpan.org
From:	Allen Smith <easmith [...] beatrice.rutgers.edu>

$Id: Tree.pm,v 1.23 2006/05/19 02:08:54 rvosa Exp $ The command to_newick uses caching: sub to_newick { my $self = shift; my @tmp = $self->_check_cache; return $tmp[1] if $tmp[0]; my $newick = unparse( -format => 'newick', -phylo => $self ); $self->_store_cache($newick); return $newick; } However, $node->set_name does not flush the cache. -Allen -- Allen Smith http://cesario.rutgers.edu/easmith/ February 1, 2003 Space Shuttle Columbia Ad Astra Per Aspera To The Stars Through Asperity

Sat Sep 16 03:35:50 2006 rutgeraldo [...] gmail.com - Correspondence added

From:

RVOSA [...] cpan.org

Thank you for bringing this to my attention. This issue hasn't been addressed yet (as of v0.15) but I will do my best to fix this soon. On Fri Sep 08 20:12:30 2006, easmith@beatrice.rutgers.edu wrote: Show quoted text

> > $Id: Tree.pm,v 1.23 2006/05/19 02:08:54 rvosa Exp $ > > The command to_newick uses caching: > > sub to_newick { > my $self = shift; > my @tmp = $self->_check_cache; > return $tmp[1] if $tmp[0]; > my $newick = unparse( -format => 'newick', -phylo => $self ); > $self->_store_cache($newick); > return $newick; > } > > However, $node->set_name does not flush the cache. > > -Allen >

Sat Sep 16 03:35:51 2006 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 20 08:37:25 2007 rutgeraldo [...] gmail.com - Correspondence added

caching will be abandoned in the upcoming v.0.16 (for which a release candidate will appear on CPAN this week). I now consider this issue resolved. Best wishes, Rutger Vos

Tue Mar 20 08:37:27 2007 rutgeraldo [...] gmail.com - Status changed from 'open' to 'resolved'

Bug #21399 for Bio-Phylo: to_newick should not use caching

Preferred bug tracker