search

coiled / feedback

A place to provide Coiled feedback
14 stars 3 forks source link

Cluster representation is ClusterBeta #188

Closed fjetter closed 1 year ago

fjetter commented 2 years ago

Really minor thing but the type of the coiled.Cluster is in fact a ClusterBeta which is also printed like that when using str/repr

In [1]: from coiled import Cluster

In [2]: Cluster.__mro__
Out[2]:
(coiled._beta.cluster.ClusterBeta,
 coiled.cluster.Cluster,
 distributed.deploy.cluster.Cluster,
 distributed.utils.SyncMethodMixin,
 typing.Generic,
 object)

In [3]: repr(Cluster)
Out[3]: "<class 'coiled._beta.cluster.ClusterBeta'>"

I think this is a remnant of the v2 rollout. I think it would be a nicer UX if the beta was not there since this is our actual production API, isn't it?

shughes-uk commented 2 years ago

Yup, sorting that out will be done at some point

fjetter commented 1 year ago

should be resolved in the next version