What is the problem?

• I cannot modify the outer_walls list in ThermalZone.outer_walls by its public @outer_walls.setter attribute in thermalzone.py line 496.

    @outer_walls.setter
    def outer_walls(self, value):
        if value is None:
            self._outer_walls = []

Why do we want to solve it?

• It is not clear if it is intentional that the lists cannot be changed. In case it should be changeable, the functions require corrections. If not, the setter function should be private or raise an error.

How do we want to solve it?

• Correct the setter functions in thermalzone.py between line 496 and 564 by adding

    if value is None:
        self._outer_walls = []
    else:
        self._outer_walls = value

or

    if value is None:
        self._outer_walls = []
    else:
        raise ValueError("'outer_walls' is immutable")