if val == 'unrolled': val = -1 # no code after the :there shouldn't be any code after the :
val = -1 should be moved to the next line
Use 4 spaces for indents.
if val == 'unrolled': val = -1 if (val >= 1 and val <= self.sides) or val == -1: self._value = va # should this be va or val?
player1 = player1dice player2 = player2diceDo these assignments on separate lines
player1 = player1dice player2 = player2dice