From 249ca9c7aa47e7bc9f5b7e1ccfcfe456c4aec0a2 Mon Sep 17 00:00:00 2001 From: filifa Date: Mon, 5 May 2025 00:09:01 -0400 Subject: [PATCH] update comments --- cmd/internal/markov/absorbing.go | 4 +++- cmd/internal/markov/dot/nodes_edges.go | 2 -- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/internal/markov/absorbing.go b/cmd/internal/markov/absorbing.go index d79bb99..f730c0c 100644 --- a/cmd/internal/markov/absorbing.go +++ b/cmd/internal/markov/absorbing.go @@ -24,7 +24,9 @@ import ( "gonum.org/v1/gonum/mat" ) -// AbsorbingMarkovChain is a graph representing an absorbing Markov chain. +// AbsorbingMarkovChain is a graph representing an absorbing Markov chain. This +// embeds a multi.WeightedDirectedGraph (as opposed to +// simple.WeightedDirectedGraph) to handle self loops. type AbsorbingMarkovChain struct { *multi.WeightedDirectedGraph } diff --git a/cmd/internal/markov/dot/nodes_edges.go b/cmd/internal/markov/dot/nodes_edges.go index fc3c7b5..c0cb9fc 100644 --- a/cmd/internal/markov/dot/nodes_edges.go +++ b/cmd/internal/markov/dot/nodes_edges.go @@ -45,8 +45,6 @@ func (n *Node) DOTID() string { // weightedLine is a DOT-aware multi.WeightedLine. By being a // multi.WeightedLine, it allows for self-loops, which are important for // absorbing Markov chains. -// TODO: this is a little confusing, maybe just have checks in the code that -// there's only one line in each edge? type weightedLine struct { multi.WeightedLine }