ManagerClient.quorum should return a namedtuple, dataclass or object

[Jackmin801]

class ManagerClient:
    def quorum(
        self, room_id: str, rank: int, step: int, checkpoint_server_addr: str
    ) -> Tuple[int, int, int, str, str, int, Optional[int], int, bool]: ...

            Ok((
                resp.quorum_id,
                resp.replica_rank,
                resp.replica_world_size,
                resp.address,
                resp.store_address,
                resp.max_step,
                resp.max_rank,
                resp.max_world_size,
                resp.heal,
            ))

This can be quite hard to read from the python side. Might be nice to wrap it in an object of sorts so the elements can be referred to by attribute name.

[Jackmin801]

Hrmm I think namedtuple or dataclass requires taking back GIL. Maybe a struct might work

[d4l3k]

Agreed -- should improve this

Right now I'm relying on pyre types to catch any issues

We could also use protobuf or use pyo3 to bind a Rust struct