docs: add DPA3 cyclohexane distillation tutorial#5564
Conversation
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (1)
⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughAdds a comprehensive knowledge distillation tutorial notebook ( ChangesKnowledge distillation tutorial notebook and documentation navigation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
66dca05 to
64dcc59
Compare
Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
64dcc59 to
053483f
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5564 +/- ##
==========================================
- Coverage 82.17% 82.14% -0.03%
==========================================
Files 898 900 +2
Lines 103576 104138 +562
Branches 4432 4473 +41
==========================================
+ Hits 85117 85548 +431
- Misses 17063 17181 +118
- Partials 1396 1409 +13 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
iProzd
left a comment
There was a problem hiding this comment.
A few issues should be fixed before merging:
-
The text repeatedly says the workflow extracts/trains on
energy,force, andviriallabels, but the generated dataset contains onlyenergy/forceand the training config sets the virial loss weights to 0. Please update the wording to avoid implying that virials are used in this case. -
Some saved notebook outputs show “3Dmol.js failed to load” with large inline JavaScript blocks. These should be cleared or replaced with static outputs before adding the notebook to the official docs.
-
The text says the Teacher MD uses 5,000 steps, but the code sets
MD_STEPS = 500. Please make these consistent.
|
I'll let the agent fix 1 and 3. Regarding 2,
3Dmol is rendered correctly on the web page. https://deepmodeling--5564.org.readthedocs.build/projects/deepmd/en/5564/getting-started/dpa3_cyclohexane_distillation.html |
Clarify that the lightweight distillation dataset uses energy and force labels only, while virial loss remains disabled, and make the Teacher MD step count match the documented 5,000 steps. Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
Align the Teacher MD prose with the existing 500-step code path so the tutorial remains quick to run. Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
| "username": "username", | ||
| "version": "5.3" | ||
| }, | ||
| "text": "Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple\nRequirement already satisfied: dpdata in /opt/mamba/lib/python3.12/site-packages (0.2.25)\nCollecting ase\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/6c/25/4f103d1bedb3593718713b3f743df7b3ff3fc68d36d6666c30265ef59c8a/ase-3.28.0-py3-none-any.whl (2.9 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m2.9/2.9 MB\u001b[0m \u001b[31m22.3 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0ma \u001b[36m0:00:01\u001b[0m\n\u001b[?25hCollecting matplotlib\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/94/95/7f522393c88313336b20d70fc849555757b2e5febc22b83b3a3f0fd4bce9/matplotlib-3.11.0-cp312-cp312-manylinux2014_x86_64.manylinux_2_17_x86_64.whl (10.0 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m10.0/10.0 MB\u001b[0m \u001b[31m60.9 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m0:01\u001b[0m\n\u001b[?25hCollecting openbabel-wheel\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/9a/a9/3e60dc173672ae40172a1f0c49fc42906d7587503f1f3acce4bd5b0cac7d/openbabel_wheel-3.1.1.23-cp312-cp312-manylinux2014_x86_64.manylinux_2_17_x86_64.whl (16.1 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m16.1/16.1 MB\u001b[0m \u001b[31m14.1 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting py3Dmol\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/8e/5a/7fd1b784a87e96e0078f49a0a13a98b4c5f644ba5597a4a3b70a2ba3e613/py3dmol-2.5.5-py2.py3-none-any.whl (7.2 kB)\nRequirement already satisfied: tqdm in /opt/mamba/lib/python3.12/site-packages (4.66.4)\nCollecting deepmd-kit[torch]\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/19/0f/ba34d3222622e44bbbe34aa5cf1326e1962204ed0d6978b0ea4ac60524c8/deepmd_kit-3.1.3-py37-none-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl (5.6 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m5.6/5.6 MB\u001b[0m \u001b[31m24.6 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hRequirement already satisfied: numpy>=1.21 in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (2.3.2)\nRequirement already satisfied: scipy in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (1.16.1)\nRequirement already satisfied: pyyaml in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (6.0.2)\nRequirement already satisfied: dargs>=0.4.7 in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (0.4.10)\nRequirement already satisfied: typing_extensions>=4.0.0 in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (4.14.1)\nRequirement already satisfied: h5py in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (3.14.0)\nRequirement already satisfied: wcmatch in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (10.1)\nRequirement already satisfied: packaging in /opt/mamba/lib/python3.12/site-packages (from deepmd-kit[torch]) (24.1)\nCollecting ml_dtypes (from deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/3a/cb/28ce52eb94390dda42599c98ea0204d74799e4d8047a0eb559b6fd648056/ml_dtypes-0.5.4-cp312-cp312-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl (5.0 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m5.0/5.0 MB\u001b[0m \u001b[31m12.3 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting mendeleev (from deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/a9/d3/c94840c7e282c18979513ac257d5099efb0a4149df933d132d75d9229158/mendeleev-1.1.0-py3-none-any.whl (1.4 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m1.4/1.4 MB\u001b[0m \u001b[31m26.2 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0ma \u001b[36m0:00:01\u001b[0m\n\u001b[?25hCollecting array-api-compat (from deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/86/16/1a8fd2b19544b84575cf84ef7aa3ad4c173b756d5f087c91f85d1b295777/array_api_compat-1.15.0-py3-none-any.whl (79 kB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m79.1/79.1 kB\u001b[0m \u001b[31m14.7 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m\n\u001b[?25hCollecting torch==2.10.0.* (from deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/b3/7a/abada41517ce0011775f0f4eacc79659bc9bc6c361e6bfe6f7052a6b9363/torch-2.10.0-3-cp312-cp312-manylinux_2_28_x86_64.whl (915.6 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m915.6/915.6 MB\u001b[0m \u001b[31m1.2 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:03\u001b[0m\n\u001b[?25hCollecting mpich (from deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/e1/ee/39ba1a669eed7bded62e686fe3d05e72943e9c8fe27b45feb79b7f3f1744/mpich-5.0.1-py3-none-manylinux_2_28_x86_64.whl (13.2 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m13.2/13.2 MB\u001b[0m \u001b[31m63.2 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting filelock (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/13/37/a065dc3bd6e49423a6532c642ca7378d3f467b1ef44c2800c937af7f9739/filelock-3.29.4-py3-none-any.whl (42 kB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m42.8/42.8 kB\u001b[0m \u001b[31m7.0 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m\n\u001b[?25hRequirement already satisfied: setuptools in /opt/mamba/lib/python3.12/site-packages (from torch==2.10.0.*->deepmd-kit[torch]) (71.0.4)\nCollecting sympy>=1.13.3 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/a2/09/77d55d46fd61b4a135c444fc97158ef34a095e5681d0a6c10b75bf356191/sympy-1.14.0-py3-none-any.whl (6.3 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m6.3/6.3 MB\u001b[0m \u001b[31m84.0 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m:00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting networkx>=2.5.1 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/9e/c9/b2622292ea83fbb4ec318f5b9ab867d0a28ab43c5717bb85b0a5f6b3b0a4/networkx-3.6.1-py3-none-any.whl (2.1 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m2.1/2.1 MB\u001b[0m \u001b[31m38.4 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m\n\u001b[?25hRequirement already satisfied: jinja2 in /opt/mamba/lib/python3.12/site-packages (from torch==2.10.0.*->deepmd-kit[torch]) (3.1.6)\nCollecting fsspec>=0.8.5 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/e5/22/4222d7ddf3da30f363edaa98e329c2bce6c65497c9cb2810931c8b2c0fbc/fsspec-2026.6.0-py3-none-any.whl (203 kB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m203.9/203.9 kB\u001b[0m \u001b[31m28.5 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m\n\u001b[?25hCollecting cuda-bindings==12.9.4 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/a9/c1/dabe88f52c3e3760d861401bb994df08f672ec893b8f7592dc91626adcf3/cuda_bindings-12.9.4-cp312-cp312-manylinux_2_24_x86_64.manylinux_2_28_x86_64.whl (12.2 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m12.2/12.2 MB\u001b[0m \u001b[31m12.5 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting nvidia-cuda-nvrtc-cu12==12.8.93 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/05/6b/32f747947df2da6994e999492ab306a903659555dddc0fbdeb9d71f75e52/nvidia_cuda_nvrtc_cu12-12.8.93-py3-none-manylinux2010_x86_64.manylinux_2_12_x86_64.whl (88.0 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m88.0/88.0 MB\u001b[0m \u001b[31m10.3 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting nvidia-cuda-runtime-cu12==12.8.90 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/0d/9b/a997b638fcd068ad6e4d53b8551a7d30fe8b404d6f1804abf1df69838932/nvidia_cuda_runtime_cu12-12.8.90-py3-none-manylinux2014_x86_64.manylinux_2_17_x86_64.whl (954 kB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m954.8/954.8 kB\u001b[0m \u001b[31m11.8 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0ma \u001b[36m0:00:01\u001b[0m\n\u001b[?25hCollecting nvidia-cuda-cupti-cu12==12.8.90 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/f8/02/2adcaa145158bf1a8295d83591d22e4103dbfd821bcaf6f3f53151ca4ffa/nvidia_cuda_cupti_cu12-12.8.90-py3-none-manylinux2014_x86_64.manylinux_2_17_x86_64.whl (10.2 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m10.2/10.2 MB\u001b[0m \u001b[31m31.8 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m0:01\u001b[0m\n\u001b[?25hCollecting nvidia-cudnn-cu12==9.10.2.21 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/ba/51/e123d997aa098c61d029f76663dedbfb9bc8dcf8c60cbd6adbe42f76d049/nvidia_cudnn_cu12-9.10.2.21-py3-none-manylinux_2_27_x86_64.whl (706.8 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m267.5/267.5 MB\u001b[0m \u001b[31m4.3 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting nvidia-cusparse-cu12==12.5.8.93 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/c2/f5/e1854cb2f2bcd4280c44736c93550cc300ff4b8c95ebe370d0aa7d2b473d/nvidia_cusparse_cu12-12.5.8.93-py3-none-manylinux2014_x86_64.manylinux_2_17_x86_64.whl (288.2 MB)\n\u001b[2K \u001b[90m━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m \u001b[32m288.2/288.2 MB\u001b[0m \u001b[31m2.6 MB/s\u001b[0m eta \u001b[36m0:00:00\u001b[0m00:01\u001b[0m00:01\u001b[0m\n\u001b[?25hCollecting nvidia-cusparselt-cu12==0.7.1 (from torch==2.10.0.*->deepmd-kit[torch])\n Downloading https://pypi.tuna.tsinghua.edu.cn/packages/56/79/12978b96bd44274fe38b5dde5cfb660b1d114f70a65ef962bcbbed99b549/nvidia_cusparselt_cu12-0.7.1-py3-none-manylinux2014_x86_64.whl (287.2 MB)\n\u001b[2K \u001b[91m━━━━━━━━━━━━━━━━━━━━━━━━━━━━\u001b[0m\u001b[91m╸\u001b[0m\u001b[90m━━━━━━━━━\u001b[0m \u001b[32m217.1/287.2 MB\u001b[0m \u001b[31m12.2 MB/s\u001b[0m eta \u001b[36m0:00:06\u001b[0mSuccessfully installed SQLAlchemy-2.0.51 annotated-types-0.7.0 array-api-compat-1.15.0 ase-3.28.0 contourpy-1.3.3 cuda-bindings-12.9.4 cuda-pathfinder-1.5.5 cycler-0.12.1 deepmd-kit-3.1.3 deprecated-1.3.1 filelock-3.29.4 fonttools-4.63.0 fsspec-2026.6.0 greenlet-3.5.2 kiwisolver-1.5.0 matplotlib-3.11.0 mendeleev-1.1.0 ml_dtypes-0.5.4 mpich-5.0.1 mpmath-1.3.0 networkx-3.6.1 nvidia-cublas-cu12-12.8.4.1 nvidia-cuda-cupti-cu12-12.8.90 nvidia-cuda-nvrtc-cu12-12.8.93 nvidia-cuda-runtime-cu12-12.8.90 nvidia-cudnn-cu12-9.10.2.21 nvidia-cufft-cu12-11.3.3.83 nvidia-cufile-cu12-1.13.1.3 nvidia-curand-cu12-10.3.9.90 nvidia-cusolver-cu12-11.7.3.90 nvidia-cusparse-cu12-12.5.8.93 nvidia-cusparselt-cu12-0.7.1 nvidia-nccl-cu12-2.27.5 nvidia-nvjitlink-cu12-12.8.93 nvidia-nvshmem-cu12-3.4.5 nvidia-nvtx-cu12-12.8.90 openbabel-wheel-3.1.1.23 pillow-12.2.0 py3Dmol-2.5.5 pydantic-2.13.4 pydantic-core-2.46.4 pyfiglet-0.8.post1 pyparsing-3.3.2 sympy-1.14.0 torch-2.10.0 triton-3.6.0 typing-inspection-0.4.2 wrapt-2.2.1\n\u001b[33mWARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv\u001b[0m\u001b[33m\n\u001b[0m" |
There was a problem hiding this comment.
The notebook is committed with all executed outputs, which should be stripped before merge (the reference quick_start.ipynb has clean outputs). Concretely: this pip install cell carries a ~53KB log full of ANSI progress bars and Tsinghua-mirror URLs (pypi.tuna.tsinghua.edu.cn), plus /opt/mamba/... paths and pinned versions; the outputs also include Bohrium/Kaggle kernel-protocol fields (parent_header, id, meta) and doubly-nested stream outputs that are not valid nbformat; and the notebook metadata retains a kaggle block and kernelspec.display_name = "exp2-dpa3-distillation". Since doc/conf.py sets nb_execution_mode = "off", this renders verbatim into the docs, and .pre-commit excludes *.ipynb so CI will not catch it. Please clear outputs and normalize the kernelspec.
There was a problem hiding this comment.
Thanks, fixed. I cleared the noisy package-install cell output, removed the non-nbformat runtime protocol fields such as execute_reply/parent_header, dropped the Kaggle metadata, and normalized the kernelspec to Python 3 (ipykernel). The remaining rendered tutorial outputs are kept so the documentation still shows the useful results and visualizations. The notebook now passes nbformat.validate.
— OpenClaw 2026.6.8 (844f405)
Clear executed notebook outputs and execution counts, remove runtime-specific metadata, and normalize the kernelspec before rendering in docs. Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
Restore rendered tutorial outputs while clearing the package-install output and removing notebook runtime protocol metadata. Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
Summary
Checks
ast.parsegit diff --checkAuthored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.5)
Summary by CodeRabbit