-
Notifications
You must be signed in to change notification settings - Fork 241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable download stats for deepforest-tree and deepforest-bird #868
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @henrykironde - left a suggestion that would be more future-proof as the library develops further and you release model checkpoints.
Note: After you've made changes, do make sure to add deepforest
tag on the Hub model repos as well, you can add them similar to how I've added them here: https://huggingface.co/weecology/deepforest-tree/discussions/1
These models are produced using Deepforest python package. https://github.com/weecology/DeepForest
Co-authored-by: Lucain <[email protected]> Co-authored-by: Vaibhav Srivastav <[email protected]>
ebbc14a
to
10fde44
Compare
d276df0
to
c3b274d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Thanks for the changes :) I see that you've already added a model to https://huggingface.co/models?other=deepforest. Feel free to do the same with the rest of the models.
@Vaibhavs10 do you want to have a last look at it before merging?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! thanks a ton for iterating on this @henrykironde 🤗
also, thanks for bringing the tag to all the other repos: https://huggingface.co/models?other=deepforest
Will merge as soon as the CI is Green ✅
@henrykironde I've checked the deepforest models and have two comments:
|
@Wauplin we do have both |
@henrykironde you can open a new PR to update the download count rule like this: countDownloads: `path_extension:"pt" OR path_extension:"pl"`, and it'll do the trick :) |
The models are produced using Deepforest, a python package.
https://github.com/weecology/DeepForest