-
Notifications
You must be signed in to change notification settings - Fork 0
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
Methode getEdge #7
Comments
Je n'avais même pas remarqué la possibilité d'utiliser la méthode Puisque le cas du multigraphe n'est pas géré pour le moment, je vais privilégier cette manière alors |
Pardon, je m'étais avancé trop vite dans la problématique de l'issue. Je ne vois pas de solution plus optimale sans l'utilisation de |
Alors, je sais que c'est moi qui t'avais dit que machin truc chose c'était utile de stocker les poids qqpart mais en réalité c'est peut être pas si utile et un: graph.getEdgeAttribute(source, target, 'weight'); Après ça risque de merder pour les graphes mixed. Je considère le rajout d'une méthode |
Je suis en train de considérer le rajout de ce genre de méthode mais il y a quand même un catch. Même avec la méthode getEdge, dans les graphe mixed tu as un soucis dans la mesure où les accesseurs à base {source,target} renvoie l'arc directed si les deux existent. Que se passe-t-il dans ton algo exactement si on y passe un graphe mixed? |
Dans le cas que tu as pointé , il n'y a pas de souci avec les graphes mixtes. --> soit il y a 1 ou 2 arcs, et alors leur poids sera rajouté s'ils sont présents |
Si c'est le cas un |
Je suis en train de rajouter une méthode |
Ici tu utilises la méthode
getEdge
qui n'existe plus dans les dernières versions degraphology
.Ses seuls cas d'usage étaient effectivement de récuperer la clé d'un arc pour lire des attributes mais comme les méthodes d'attributs peuvent prendre des combo
{source,target}
il a été décidé de les enlever parce qu'elle sont curieuses: elles ne marchent pas sur un graph multi et il n'existe pas de méthodegetNode
.Mais ton cas est interessant là et demande peut-être qu'on réattaque la question sous un angle différent.
The text was updated successfully, but these errors were encountered: