Skip Menu |

This queue is for tickets about the Graph-Easy CPAN distribution.

Report information
The Basics
Id: 104723
Status: open
Priority: 0/
Queue: Graph-Easy

People
Owner: Nobody in particular
Requestors: user42_kevin [...] yahoo.com.au
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: has_edge() arg order for undirected graph
Date: Wed, 27 May 2015 08:48:57 +1000
To: bug-Graph-Easy [...] rt.cpan.org
From: Kevin Ryde <user42_kevin [...] yahoo.com.au>
The docs of has_edge() say it is for compatibility with Graph.pm. On an undirected graph, Graph.pm has_edge() is true of vertex names either way around. But Graph::Easy has_edge() is only true of vertex names their original way around. foo.pl below prints Graph 0.96 undirected=yes yes yes Graph::Easy 0.75 undirected=yes yes no Is this meant to be? If so the Graph::Easy docs could note an undirected graph has_edge() answers to the vertex order an edge was added (or whatever the rule).

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

Subject: Re: [rt.cpan.org #104723] has_edge() arg order for undirected graph
Date: Fri, 29 May 2015 03:40:49 -0400
To: bug-Graph-Easy [...] rt.cpan.org
From: "Tels" <nospam-abuse [...] bloodgate.com>
Hello Kevin, On Tue, May 26, 2015 6:51 pm, Kevin Ryde via RT wrote: Show quoted text
> The docs of has_edge() say it is for compatibility with Graph.pm. On an > undirected graph, Graph.pm has_edge() is true of vertex names either way > around. But Graph::Easy has_edge() is only true of vertex names their > original way around. foo.pl below prints > > Graph 0.96 undirected=yes > yes > yes > Graph::Easy 0.75 undirected=yes > yes > no > > Is this meant to be? If so the Graph::Easy docs could note an > undirected graph has_edge() answers to the vertex order an edge was > added (or whatever the rule).
IMO It should return yes in both cases, being compatible and all that. best regards, tels