BaseLoader needs str/repr dunder funcs

dmarx / Multi-Modal-Comparators

Unified API to facilitate usage of pre-trained "perceptor" models, a la CLIP

39 stars 4 forks source link

BaseLoader needs str/repr dunder funcs #8

Closed dmarx closed 2 years ago

dmarx commented 2 years ago

specifically, this is less helpful than it could be:

mmc.REGISTRY.find(architecture='clip')

[<mmc.loaders.openaicliploader.OpenAiClipLoader at 0x7fc1f90f3150>,
 <mmc.loaders.openaicliploader.OpenAiClipLoader at 0x7fc0f0a9dfd0>,
 <mmc.loaders.openaicliploader.OpenAiClipLoader at 0x7fc0f0a9da50>,
...
]

dmarx commented 2 years ago

looks like __str__ already defined, guess __repr__ is the culprit here.