gambit icon indicating copy to clipboard operation
gambit copied to clipboard

ENH: action values for zero-probability infosets should be undefined

Open rahulsavani opened this issue 2 years ago • 2 comments

Currently, the action value (stored in map_actionsValues) for actions at zero-probability information sets are zero. This enhancement would instead have these recorded as undefined. This may require changing the type of values of the map which is currently, via a template, either a double of Gambit.Rational, or just not populating it for the respective actions at zero-prob infosets.

rahulsavani avatar Feb 11 '24 08:02 rahulsavani

We'll also need to give some consideration about the best way to do this, given that C++ does not have a natural "null" value. One possibility is to raise an exception, though this will take a bit of care to ensure code that calls this currently is ready for that.

tturocy avatar Feb 11 '24 11:02 tturocy