Skip to content

update arx5 asset#192

Merged
matafela merged 1 commit intomainfrom
cj/update-arx5-asset
Mar 20, 2026
Merged

update arx5 asset#192
matafela merged 1 commit intomainfrom
cj/update-arx5-asset

Conversation

@matafela
Copy link
Collaborator

Description

Update arx5 asset.

Type of change

  • Enhancement (non-breaking change which improves an existing functionality)

Checklist

  • I have run the black . command to format the code base.
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Dependencies have been updated, if applicable.

Copilot AI review requested due to automatic review settings March 20, 2026 07:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the ARX5 robot asset reference used by the EmbodiChain dataset loader to point at a newer ZIP and checksum, so consumers download/verify the updated asset bundle.

Changes:

  • Update ARX5 asset archive name from ARX5.zip to ARX5_2.zip.
  • Update the corresponding checksum used for asset integrity verification.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +557 to +558
os.path.join(EMBODICHAIN_DOWNLOAD_PREFIX, robot_assets, "ARX5_2.zip"),
"6fc1d6a5a35b7926ba6325c0d20f5bc3",
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

PR description/checklist states that documentation changes and tests were added, but this PR diff only updates the ARX5 asset URL/checksum. Either include the related doc/test updates in this PR, or adjust the PR description/checklist to match the actual changes.

Copilot uses AI. Check for mistakes.
@matafela matafela merged commit a72d2db into main Mar 20, 2026
9 checks passed
@matafela matafela deleted the cj/update-arx5-asset branch March 20, 2026 07:38
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.

3 participants