Skip to content

Chore: update DPA-2.4-7M ckpt#5496

Queued
anyangml wants to merge 2 commits into
deepmodeling:masterfrom
anyangml:chore/update-dpa-2_4_7M
Queued

Chore: update DPA-2.4-7M ckpt#5496
anyangml wants to merge 2 commits into
deepmodeling:masterfrom
anyangml:chore/update-dpa-2_4_7M

Conversation

@anyangml
Copy link
Copy Markdown
Collaborator

@anyangml anyangml commented Jun 5, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Updated the pre-trained DPA-2.4-7M model to the patched release.
    • Model artifact checksum updated and downloads now use additional mirrored sources to improve retrieval reliability.

Copilot AI review requested due to automatic review settings June 5, 2026 02:08
@github-actions github-actions Bot added the Python label Jun 5, 2026
@anyangml anyangml requested a review from iProzd June 5, 2026 02:08
Copy link
Copy Markdown
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 built-in pretrained-model registry to point the DPA-2.4-7M entry at a new checkpoint artifact (URLs + checksum), so users pulling pretrained models via deepmd.pretrained will fetch the updated weights.

Changes:

  • Switched DPA-2.4-7M download URLs to the new DPA-2.4-7M-patched-mt.pt artifact.
  • Updated the registry’s filename and sha256 for DPA-2.4-7M to match the new checkpoint.

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

Comment thread deepmd/pretrained/registry.py Outdated
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 5, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f9fd1516-8bb1-4085-8a13-8668e02f740c

📥 Commits

Reviewing files that changed from the base of the PR and between 9a1421c and eaebd5d.

📒 Files selected for processing (1)
  • deepmd/pretrained/registry.py
🚧 Files skipped from review as they are similar to previous changes (1)
  • deepmd/pretrained/registry.py

📝 Walkthrough

Walkthrough

The PR updates the built-in pretrained model registry entry for DPA-2.4-7M to reference the patched checkpoint (DPA-2.4-7M-patched-mt.pt) by changing its download URLs and replacing the stored SHA256 checksum; the registry filename remains dpa-2.4-7M.pt.

Changes

DPA-2.4-7M Registry Update

Layer / File(s) Summary
Update DPA-2.4-7M model registry entry
deepmd/pretrained/registry.py
Updated the MODEL_REGISTRY["DPA-2.4-7M"] entry: download URLs now reference DPA-2.4-7M-patched-mt.pt on Hugging Face, hf-mirror, and modelscope, and the sha256 checksum value was changed; the registry filename remains dpa-2.4-7M.pt.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

Python

Suggested reviewers

  • iProzd
  • njzjz
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Chore: update DPA-2.4-7M ckpt' directly and specifically refers to the main change in the pull request: updating the DPA-2.4-7M checkpoint entry in the model registry.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Anyang Peng <137014849+anyangml@users.noreply.github.com>
@anyangml anyangml enabled auto-merge June 5, 2026 02:16
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.37%. Comparing base (27a18b6) to head (eaebd5d).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5496      +/-   ##
==========================================
+ Coverage   81.36%   81.37%   +0.01%     
==========================================
  Files         868      868              
  Lines       96567    96598      +31     
  Branches     4233     4240       +7     
==========================================
+ Hits        78570    78611      +41     
+ Misses      16697    16684      -13     
- Partials     1300     1303       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@anyangml anyangml added this pull request to the merge queue Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants