DecisionProgramming.jl icon indicating copy to clipboard operation
DecisionProgramming.jl copied to clipboard

Wrong names in setindex-functions

Open solliolli opened this issue 9 months ago • 1 comments

The setindex-functions for probability and utility matrices throw an error if a non-existent state is given, but all the functions have probability_matrix.nodes[i] when it should be PM or UM, depending on the function.

  • Probability matrices: https://github.com/gamma-opt/DecisionProgramming.jl/blob/d31a5277b8f315a1b3db5cc746fd45787964be25/src/influence_diagram.jl#L570-L599
  • Utility matrices: https://github.com/gamma-opt/DecisionProgramming.jl/blob/d31a5277b8f315a1b3db5cc746fd45787964be25/src/influence_diagram.jl#L706-L735

solliolli avatar May 16 '24 07:05 solliolli