Skip to content

madengine v2: ROCm path override + RPD e2e dependency fix#76

Merged
coketaste merged 4 commits intocoketaste/refactor-disfrom
coketaste/refactor-dis-rocm-path
Mar 1, 2026
Merged

madengine v2: ROCm path override + RPD e2e dependency fix#76
coketaste merged 4 commits intocoketaste/refactor-disfrom
coketaste/refactor-dis-rocm-path

Conversation

@coketaste
Copy link
Copy Markdown
Collaborator

feat(madengine): ROCm path override + RPD e2e dependency fix

  • Add get_rocm_path() and --rocm-path / ROCM_PATH support for madengine run
    • constants: get_rocm_path(override) with env and default /opt/rocm
    • Context, ROCmToolManager, gpu_tool_factory, gpu_validator, container_runner
      and run orchestrator use configurable rocm_path
    • CLI: madengine run --rocm-path for non-default ROCm installs (e.g. Rock)
  • Tests: unit tests for get_rocm_path, Context rocm_path, ROCmToolManager paths,
    run --help; fixtures use get_rocm_path() for AMD detection; integration
    test_get_cached_managers updated for (vendor, rocm_path) cache keys
  • Fix RPD profiling e2e: install nlohmann-json3-dev (Ubuntu) and json-devel
    (CentOS) in trace.sh so rpd_tracer builds in container

ROCM_PATH / --rocm-path:
- Add get_rocm_path() and wire through Context, ROCmToolManager, gpu_tool_factory,
  gpu_validator, container_runner, run orchestrator; add --rocm-path to run CLI
- Unit tests for get_rocm_path, Context, ROCmToolManager, run --help; update
  fixtures and test_get_cached_managers for (vendor, rocm_path) cache

RPD e2e:
- pre_scripts/trace.sh: install nlohmann-json3-dev (Ubuntu) and json-devel
  (CentOS) so rocmProfileData rpd_tracer build finds nlohmann/json.hpp
@coketaste coketaste self-assigned this Mar 1, 2026
@coketaste coketaste changed the title ROCm path override + RPD e2e dependency fix madengine v2: ROCm path override + RPD e2e dependency fix Mar 1, 2026
@coketaste coketaste changed the base branch from main to coketaste/refactor-dis March 1, 2026 04:01
@coketaste coketaste merged commit 8d292d0 into coketaste/refactor-dis Mar 1, 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.

1 participant