Skip to content

Add per-tool dependency documentation#66

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/document-dependencies-for-tools
Draft

Add per-tool dependency documentation#66
Copilot wants to merge 3 commits intomainfrom
copilot/document-dependencies-for-tools

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

No single reference existed for what each IntelliKit tool requires — Python packages, system tools, and hidden system dependencies were scattered across individual READMEs and build files.

Changes

  • docs/dependencies.md — New dependency reference covering all six tools, focused on dependencies that must be manually installed:

    • System tool requirements (rocprofv3, rocminfo, hipcc, AMDuProfCLI)
    • Hidden system dependencies of kernelDB (auto-fetched from GitHub): libdwarf (required — fatal CMake error if missing), libelf, libzstd, amd_comgr, and libhsa-runtime64
    • Install commands for system packages (apt/dnf)
    • External dependency details for ROCm and AMD uProf
    • Full dependency graph
  • README.md — Added a link to docs/dependencies.md in the Documentation section

Auto-fetched C++ libraries (spdlog, fmt, nlohmann/json via CMake FetchContent/CPM) are intentionally excluded as they require no manual installation.

Dependency Graph

intellikit
├── accordo   → numpy, mcp[cli], kerneldb (auto-fetched); System: ROCm >= 6.0, CMake >= 3.22, libdwarf, libelf, libzstd
├── linex     → mcp[cli]; System: rocprofv3 (ROCm 7.0+)
├── metrix    → pandas>=1.5.0, mcp[cli]; System: rocprofv3, rocminfo (ROCm 6.x+)
├── nexus     → mcp[cli]; System: ROCm >= 6.0, LLVM (from ROCm), CMake >= 3.22, libdwarf, libelf, libzstd
├── rocm_mcp  → mcp[cli]>=1.21.0, beautifulsoup4, httpx; System: hipcc, rocminfo
└── uprof_mcp → mcp[cli]>=1.21.0; System: AMDuProfCLI
Original prompt

This section details on the original issue you should resolve

<issue_title>Document dependancies</issue_title>
<issue_description>Document the dependancies we need for the tools and their dependencies (e.g., kernelDB or others). Please inspect all code, clone dependencies and document the per tool dependencies in markdown.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: mawad-amd <112003944+mawad-amd@users.noreply.github.com>
Copilot AI changed the title [WIP] Document dependencies for tools in markdown Add per-tool dependency documentation Mar 5, 2026
@mawad-amd
Copy link
Member

@copilot we don't need to document dependencies that get pulled automatically. Also you missed libdwarf which is a kerneldb dependency. Make sure you go through the CMake files of the dependencies pulled from GitHub and make sure these hidden dependencies are captured.

…libdwarf, libelf, libzstd)

Co-authored-by: mawad-amd <112003944+mawad-amd@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 9, 2026

@copilot we don't need to document dependencies that get pulled automatically. Also you missed libdwarf which is a kerneldb dependency. Make sure you go through the CMake files of the dependencies pul...

Done in commit 7a95486. Changes made:

  • Removed all auto-fetched C++ library sections (spdlog, fmt, nlohmann/json via CMake FetchContent/CPM)
  • Inspected kernelDB's CMakeLists.txt and src/CMakeLists.txt at the pinned commit. It uses find_path for libdwarf headers (fatal CMake error if missing) and links against dwarf, elf, zstd, and amd_comgr
  • Added all kernelDB hidden system dependencies to both Accordo and Nexus sections: libdwarf, libelf, libzstd (with apt/dnf install commands), plus amd_comgr and libhsa-runtime64 (both from ROCm, no separate install needed)
  • Document now only covers what must be manually installed

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/AMDResearch/kerneldb/git/trees/6e0093972be276a51ed2bea963c756caa9324325
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

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.

Document dependancies

2 participants