Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix cut sharing in a graph with zero-probability arcs #797

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
+78 −12
Open

Fix cut sharing in a graph with zero-probability arcs #797

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
6ca11ac
Fix cut sharing in a graph with zero-probability arcs
odow Oct 23, 2024
f921657
Update src/algorithm.jl
odow Oct 23, 2024
faccaae
Update
odow Oct 25, 2024
c131148
Update
odow Oct 25, 2024
0cd9a0b
Update
odow Oct 25, 2024
652ade7
Apply suggestions from code review
odow Oct 25, 2024
94d0a7f
Update
odow Oct 25, 2024
33cd191
Update
odow Oct 25, 2024
68b40de
Merge branch 'master' into od/same-children
odow Nov 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion src/algorithm.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ end
function get_same_children(model::PolicyGraph{T}) where {T}
tmp = Dict{Set{T},Set{T}}()
for (key, node) in model.nodes
children = Set(child.term for child in node.children)
children =
Set(child.term for child in node.children if child.probabillity > 0)
odow marked this conversation as resolved.
Show resolved Hide resolved
if length(children) == 0
continue
elseif haskey(tmp, children)
Expand Down
Loading