Clarify automated review feedback on MpiGroup.__del__ implementation

[Copilot]

The automated reviewer flagged the MpiGroup.__del__ method as "overly complex" in PR #698. This assessment was incorrect.

Analysis

The __del__ method implements standard defensive cleanup for MPI communicators:

def __del__(self):
    if self.comm != MPI.COMM_NULL and MPI.Is_initialized() and not MPI.Is_finalized():
        self.comm.Free()

The three conditions are necessary for safe MPI resource cleanup:

• self.comm != MPI.COMM_NULL: Prevents freeing null communicators
• MPI.Is_initialized(): Guards against calling MPI functions before initialization
• not MPI.Is_finalized(): Prevents use-after-finalize errors

This pattern is required in Python where destructor ordering is non-deterministic. No changes needed.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.