Allow to retrieve discussions/PRs using list_models

It would be nice to have a discussions=True parameter in list_models and other listing methods, to also add the discussions related information in the response. At the moment, one needs to do list_models + get_repo_discussions for each, but that involves quite a bit of work.

@julien-c argument on keeping as they are was

config and cardData are properties of the model in the data backend, whereas discussions are another object completely

From my side, I don’t think that’s a distinction that users should know/internalize, and I would find making list_models more powerful a better approach from a user perspective

(internal discussion)

WDYT @julien-c @Pierrci @Wauplin

[Wauplin]

Coding the helper in huggingface_hub directly is a bit clunky in my opinion. If the helper is just a for loop to get the discussions repo after repo, we end up with potentially thousands of calls to the API and a very long execution time

models = list_models()
for model in models:
    model.discussions = list(get_repo_discussions(model.id))

Even if we parallelize the calls, it still seems very unefficient.

[julien-c]

an API always reflect a bit the underlying data model, IMO

I’d be in favor of keeping the server API unchanged, but coding a helper in huggingface_hub could be an option (I thought you were suggesting changing the server API)