Should we update syntax for loading model cards?

nateraw / modelcards

📝 Utility to create, edit, and publish model cards on the Hugging Face Hub. [**Now lives in huggingface_hub**]

MIT License

15 stars 4 forks source link

Closed nateraw closed 2 years ago

nateraw commented 2 years ago

So right now, I have it so ModelCard.load works for both repo IDs and local files.

I'm thinking perhaps we could do:

ModelCard.open(filepath) to open a local file
ModelCard.load_from_hub(repo_id) to load a remote Hugging Face model repo model card

WDYT?

julien-c commented 2 years ago

.load() is fine IMO

nateraw commented 2 years ago

Thank you for the feedback!! Its nice to not talk to yourself...