Skip to content

fix: pin NequIP to legacy API#125

Closed
njzjz-bot wants to merge 1 commit into
deepmodeling:pt-expt-dense-edge-opfrom
njzjz-bot:fix-pr-124-nequip-pin
Closed

fix: pin NequIP to legacy API#125
njzjz-bot wants to merge 1 commit into
deepmodeling:pt-expt-dense-edge-opfrom
njzjz-bot:fix-pr-124-nequip-pin

Conversation

@njzjz-bot

Copy link
Copy Markdown
Contributor

Fixes the CUDA test collection failure:

ImportError: cannot import name 'model_from_config' from 'nequip.model'

deepmd_gnn.nequip still uses the pre-0.7 NequIP API (nequip.model.model_from_config). NequIP 0.7+ removed that symbol, while the CI dependency setup already pins/overrides e3nn==0.4.4, which is compatible with NequIP 0.6.x. This PR pins NequIP to <0.7 consistently in runtime metadata, nox, and the CUDA workflow.

Verification:

  • git diff --check
  • python3 -m py_compile noxfile.py
  • uv pip install --dry-run --overrides requirements-overrides.txt ... "nequip<0.7" "e3nn==0.4.4" ... --torch-backend auto resolves nequip==0.6.2

Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)

NequIP 0.7 removed nequip.model.model_from_config, which this package still imports. Keep the runtime/test dependency below 0.7 so CI resolves nequip 0.6.x with the existing e3nn 0.4.4 override.

Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
@codecov

codecov Bot commented Jun 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.06%. Comparing base (4d6071a) to head (4fe334e).

Additional details and impacted files
@@                  Coverage Diff                   @@
##           pt-expt-dense-edge-op     #125   +/-   ##
======================================================
  Coverage                  82.06%   82.06%           
======================================================
  Files                         12       12           
  Lines                       1299     1299           
======================================================
  Hits                        1066     1066           
  Misses                       233      233           

☔ 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.

@njzjz njzjz deleted the branch deepmodeling:pt-expt-dense-edge-op June 21, 2026 18:37
@njzjz njzjz closed this Jun 21, 2026
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