Skip to content

miss rdkit for intern-s models#4587

Open
lvhan028 wants to merge 1 commit into
InternLM:mainfrom
lvhan028:rkdit-deps
Open

miss rdkit for intern-s models#4587
lvhan028 wants to merge 1 commit into
InternLM:mainfrom
lvhan028:rkdit-deps

Conversation

@lvhan028
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 14, 2026 09:09
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

This PR adds RDKit to runtime dependency lists so Intern-S models can access the chemistry toolkit dependency across supported target-device installs.

Changes:

  • Adds RDkit to CUDA, ROCm, MACA, CAMB, and Ascend runtime requirements.
  • Keeps the dependency positioned near other alphabetized runtime packages.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
requirements/runtime_cuda.txt Adds RDKit to CUDA runtime dependencies.
requirements/runtime_rocm.txt Adds RDKit to ROCm runtime dependencies.
requirements/runtime_maca.txt Adds RDKit to MACA runtime dependencies.
requirements/runtime_camb.txt Adds RDKit to CAMB runtime dependencies.
requirements/runtime_ascend.txt Adds RDKit to Ascend runtime dependencies.

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

Comment thread requirements/runtime_cuda.txt Outdated
Comment thread requirements/runtime_rocm.txt Outdated
Comment thread requirements/runtime_maca.txt Outdated
Comment thread requirements/runtime_camb.txt Outdated
Comment thread requirements/runtime_ascend.txt Outdated
@lvhan028 lvhan028 changed the title miss RDkit for intern-s models miss rdkit for intern-s models May 14, 2026
@lvhan028 lvhan028 requested a review from zhulinJulia24 May 15, 2026 02:01
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