Skip Menu |

This queue is for tickets about the Chess-Rep CPAN distribution.

Report information
The Basics
Id: 53929
Status: resolved
Priority: 0/
Queue: Chess-Rep
People
Owner: Nobody in particular
Requestors: olivier.macchioni [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Jan 22 22:46:17 2010 olivier.macchioni [...] gmail.com - Ticket created
Subject: Problem when computing the SAN moves - disambiguation fails
Date: Sat, 23 Jan 2010 04:45:41 +0100
To: bug-Chess-Rep [...] rt.cpan.org
From: Olivier Macchioni <olivier.macchioni [...] gmail.com>
This is a sample code which triggers the problem: 8<------------------------------ #!/usr/bin/perl -w use Data::Dumper; use Chess::Rep; my $pos = Chess::Rep->new('3R4/p4r1k/1p4r1/2p1Pp1p/5P2/P6P/1P3K2/3R4 w - -'); my $h = $pos->go_move('d8d7'); print Dumper($h); 8<------------------------------ The result for $h->{'san'} is Rd8d7 although I beleive it should be R8d7, which is the shortest non-ambiguous notation, hence the valid SAN representation of the move.
  Mon Jan 25 16:05:41 2010 olivier.macchioni [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #53929] AutoReply: Problem when computing the SAN moves - disambiguation fails
Date: Mon, 25 Jan 2010 22:04:49 +0100
To: bug-Chess-Rep [...] rt.cpan.org
From: Olivier Macchioni <olivier.macchioni [...] gmail.com>
The patch attached is to be used vs version 0.5 I have added a few tests cases regarding disambiguation and I beleive I have fixed the problem - feedback welcome, Mac

Message body is not shown because sender requested not to inline it.

  Mon Jan 25 16:37:01 2010 olivier.macchioni [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #53929] AutoReply: Problem when computing the SAN moves - disambiguation fails
Date: Mon, 25 Jan 2010 22:04:49 +0100
To: bug-Chess-Rep [...] rt.cpan.org
From: Olivier Macchioni <olivier.macchioni [...] gmail.com>
The patch attached is to be used vs version 0.5 I have added a few tests cases regarding disambiguation and I beleive I have fixed the problem - feedback welcome, Mac

Message body is not shown because sender requested not to inline it.

  Tue Jan 26 06:53:25 2010 mihai.bazon [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #53929] AutoReply: Problem when computing the SAN moves - disambiguation fails
Date: Tue, 26 Jan 2010 13:52:55 +0200
To: bug-Chess-Rep [...] rt.cpan.org
From: Mihai Călin Bazon <mihai.bazon [...] gmail.com>
Hi Olivier, Your patch seems fine. Thanks for taking the time to create it! I applied it to my source tree. Not sure if I should release a new version just for this—what do you think? It seems like a cosmetic issue to me. Cheers, -Mihai On Mon, Jan 25, 2010 at 11:37 PM, Olivier Macchioni via RT < bug-Chess-Rep@rt.cpan.org> wrote: Show quoted text
> Queue: Chess-Rep > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > The patch attached is to be used vs version 0.5 > > I have added a few tests cases regarding disambiguation and I beleive I > have > fixed the problem - feedback welcome, > > Mac > >
-- Mihai Bazon, http://mihai.bazon.net/blog
  Tue Jan 26 06:53:27 2010 The RT System itself - Status changed from 'new' to 'open'
  Tue Jan 26 07:00:26 2010 olivier.macchioni [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #53929] AutoReply: Problem when computing the SAN moves - disambiguation fails
Date: Tue, 26 Jan 2010 13:00:06 +0100
To: bug-Chess-Rep [...] rt.cpan.org
From: Olivier Macchioni <olivier.macchioni [...] gmail.com>
Hi Mihai, Thanks for the follow-up. It was not a cosmetic issue at all to me... I'm writing software which automatically runs various chess engines to analyze games for a well-known chess website. I needed to indirectly compute the transform SAN -> Long Algebraic -> SAN... and when the initial SAN and the final SAN don't match, it's really bad.. I don't know if you want to release a new version - that's not really my call... but of course it would make my life easier when providing the installation instructions to my customer :P Olivier On Tue, Jan 26, 2010 at 12:53 PM, Mihai Bazon via RT < bug-Chess-Rep@rt.cpan.org> wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > Hi Olivier, > > Your patch seems fine. Thanks for taking the time to create it! > > I applied it to my source tree. Not sure if I should release a new version > just for this—what do you think? It seems like a cosmetic issue to me. > > Cheers, > -Mihai > > On Mon, Jan 25, 2010 at 11:37 PM, Olivier Macchioni via RT < > bug-Chess-Rep@rt.cpan.org> wrote: >
> > Queue: Chess-Rep > > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > > > The patch attached is to be used vs version 0.5 > > > > I have added a few tests cases regarding disambiguation and I beleive I > > have > > fixed the problem - feedback welcome, > > > > Mac > > > >
> > > -- > Mihai Bazon, > http://mihai.bazon.net/blog > >
  Tue Jan 26 07:24:49 2010 mihai.bazon [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #53929] AutoReply: Problem when computing the SAN moves - disambiguation fails
Date: Tue, 26 Jan 2010 14:24:25 +0200
To: bug-Chess-Rep [...] rt.cpan.org
From: Mihai Călin Bazon <mihai.bazon [...] gmail.com>
I submitted version 0.6 with this patch. Hope all is fine. Cheers, -Mihai On Tue, Jan 26, 2010 at 2:00 PM, Olivier Macchioni via RT < bug-Chess-Rep@rt.cpan.org> wrote: Show quoted text
> Queue: Chess-Rep > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > Hi Mihai, > > Thanks for the follow-up. > > It was not a cosmetic issue at all to me... I'm writing software which > automatically runs various chess engines to analyze games for a well-known > chess website. > > I needed to indirectly compute the transform SAN -> Long Algebraic -> > SAN... > and when the initial SAN and the final SAN don't match, it's really bad.. > > I don't know if you want to release a new version - that's not really my > call... but of course it would make my life easier when providing the > installation instructions to my customer :P > > Olivier > > > > > On Tue, Jan 26, 2010 at 12:53 PM, Mihai Bazon via RT < > bug-Chess-Rep@rt.cpan.org> wrote: >
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > > > Hi Olivier, > > > > Your patch seems fine. Thanks for taking the time to create it! > > > > I applied it to my source tree. Not sure if I should release a new
> version
> > just for this—what do you think? It seems like a cosmetic issue to me. > > > > Cheers, > > -Mihai > > > > On Mon, Jan 25, 2010 at 11:37 PM, Olivier Macchioni via RT < > > bug-Chess-Rep@rt.cpan.org> wrote: > >
> > > Queue: Chess-Rep > > > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=53929 > > > > > > > The patch attached is to be used vs version 0.5 > > > > > > I have added a few tests cases regarding disambiguation and I beleive I > > > have > > > fixed the problem - feedback welcome, > > > > > > Mac > > > > > >
> > > > > > -- > > Mihai Bazon, > > http://mihai.bazon.net/blog > > > >
> >
-- Mihai Bazon, http://mihai.bazon.net/blog
  Tue Feb 02 15:57:24 2010 mihai.bazon [...] gmail.com - Correspondence added
Fixed in 0.6
  Tue Feb 02 15:57:27 2010 mihai.bazon [...] gmail.com - Status changed from 'open' to 'resolved'