allenai / allenai/catwalk

Request: Either remove `short_name_for_model_object` or allow for full name registering of models

Open
#116 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
154
Forks
18
PR merge metrics
No merged PRs in 30d

Description

Took a while to debug why `google/t5-v1_1-small` wasn't working even though it's registered in `models/__init__.py`. It's not obvious how the shortened name is particularly beneficial whereas the cost is that it's hard to know the right flags for model names to pass to catwalk.

Recommend either removing this shortener or at least always supporting the full name of the model.

Haven't tested this for Tasks, but I imagine similar type of issue.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.