Skip to content

Add HoloMotion library integration with ONNX download counting#2109

Merged
Wauplin merged 2 commits intohuggingface:mainfrom
raymond-myc:raymond-myc/holomotion-library-integration
Apr 27, 2026
Merged

Add HoloMotion library integration with ONNX download counting#2109
Wauplin merged 2 commits intohuggingface:mainfrom
raymond-myc:raymond-myc/holomotion-library-integration

Conversation

@raymond-myc
Copy link
Copy Markdown
Contributor

@raymond-myc raymond-myc commented Apr 17, 2026

This PR registers holomotion as a Hub library integration.

Why:

  • HoloMotion models are distributed primarily as ONNX artifacts.
  • We want Hub download metrics to reflect direct ONNX file requests.

Changes:

  • Add holomotion entry to model-libraries.ts
  • Set:
    • prettyLabel: HoloMotion
    • repoName: HoloMotion
    • repoUrl: https://github.com/HorizonRobotics/HoloMotion
    • filter: false
    • countDownloads: path_extension:"onnx"

Related model:

  • HorizonRobotics/HoloMotion_v1.2

The model card for that repo has been updated to include:

library_name: holomotion
tags:
  - onnx

This should allow direct ONNX requests to contribute to the model’s public download count.

Made with Cursor


Note

Low Risk
Single additive config entry that only affects library metadata and download-count query matching for models tagged holomotion.

Overview
Registers a new Hub model library integration holomotion in model-libraries.ts, including label/repo metadata pointing to HorizonRobotics/HoloMotion.

Configures countDownloads for this library to count ONNX artifact requests via the Elastic query path_extension:"onnx", so ONNX downloads contribute to model download metrics.

Reviewed by Cursor Bugbot for commit ef75621. Bugbot is set up for automated code reviews on this repo. Configure here.

Register holomotion as a Hub library so ONNX artifact requests can contribute to repo-level download counts.

Made-with: Cursor
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 337f865. Configure here.

repoUrl: "https://github.com/HorizonRobotics/HoloMotion",
filter: false,
countDownloads: `path_extension:"onnx"`,
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New entry breaks alphabetical ordering of library map

Low Severity

The holomotion entry is inserted between hermes and hezar, but alphabetically "hezar" (he-z) comes before "holomotion" (ho-l) since "he" < "ho". The entry needs to be placed after hezar and before htrflow to maintain the file's alphabetical ordering convention.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 337f865. Configure here.

Copy link
Copy Markdown
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is correct but requires https://huggingface.co/HorizonRobotics/HoloMotion_v1.2 to be correctly tagged before getting this merged (we need https://huggingface.co/models?other=holomotion to be populated)

@raymond-myc
Copy link
Copy Markdown
Contributor Author

This PR is correct but requires https://huggingface.co/HorizonRobotics/HoloMotion_v1.2 to be correctly tagged before getting this merged (we need https://huggingface.co/models?other=holomotion to be populated)

Thanks for checking out.
https://huggingface.co/HorizonRobotics/HoloMotion_v1.2/blob/main/README.md
Is the current form correctly tagged ?

@Wauplin
Copy link
Copy Markdown
Contributor

Wauplin commented Apr 27, 2026

This PR is correct but requires https://huggingface.co/HorizonRobotics/HoloMotion_v1.2 to be correctly tagged before getting this merged (we need https://huggingface.co/models?other=holomotion to be populated)

Thanks for checking out. https://huggingface.co/HorizonRobotics/HoloMotion_v1.2/blob/main/README.md Is the current form correctly tagged ?

Yes! As you can see it here, model now correctly shows up: https://huggingface.co/models?other=holomotion

I'll merge this PR, expect 2-3 days before getting it live on the Hub 🤗

@Wauplin Wauplin merged commit cc4f181 into huggingface:main Apr 27, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants