You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is not exactly a feature request, I guess, but hopefully still something along those lines?
Is your feature request related to a problem? Please describe.
Multiple files such as timm/models/mobilenetv3.py are missing type hints and docs for what the various parameters mean.
Describe the solution you'd like
Addition of docs where necessary and improvement in type hints.
Describe alternatives you've considered
.
Additional context
As someone who is trying to understand paper-to-code implementations of different models, I find it helpful to have descriptions of uncommon parameters. It's also helpful to have type hints on the return types and consumed parameters of functions, which really improve the behaviour of static type analysis tools in type-ahead suggestions made.
Is this something the team/community would be interested in having added? If so, I would love to try and contribute here. I'm trying to do something similar on the diffusers repository as well in my free time.
The text was updated successfully, but these errors were encountered:
@a-r-r-o-w so yeah, there are a LOT of missing type hints and docstrings, some more recent models have some, most older ones do not.
I'm open to PRs, as it'll be nice to improve this. But, incorrect / misleading typing and comments are worse than none, so will be somewhat picky.
For docstrings, should try to adhere to Google style. No redundancy of type or default info from argument definitions in the docstring, as that's too difficult to maintain.
This is not exactly a feature request, I guess, but hopefully still something along those lines?
Is your feature request related to a problem? Please describe.
Multiple files such as
timm/models/mobilenetv3.py
are missing type hints and docs for what the various parameters mean.Describe the solution you'd like
Addition of docs where necessary and improvement in type hints.
Describe alternatives you've considered
.
Additional context
As someone who is trying to understand paper-to-code implementations of different models, I find it helpful to have descriptions of uncommon parameters. It's also helpful to have type hints on the return types and consumed parameters of functions, which really improve the behaviour of static type analysis tools in type-ahead suggestions made.
Is this something the team/community would be interested in having added? If so, I would love to try and contribute here. I'm trying to do something similar on the diffusers repository as well in my free time.
The text was updated successfully, but these errors were encountered: