Repository navigation
Federate the six rocprof-compute skills - #285
Draft
feizheng10 wants to merge 3 commits into
Draft
feizheng10 wants to merge 3 commits into
feizheng10 wants to merge 3 commits into
Conversation
The rocprofiler-compute project is already approved, and these skills stay owned on that repository's develop branch. Co-authored-by: Cursor <cursoragent@cursor.com>
feizheng10
marked this pull request as draft
October 26, 2026 15:47
2 of 3 tasks
develop does not have those eval changes yet, so this import uses that pull request's head and keeps each skill's machine.yml here, where skillscope selects the MI300 runner. Co-authored-by: Cursor <cursoragent@cursor.com>
Federation copies skill folders only, so ../../sample does not exist here. A fixture entry copies that tree once and retargets the vendored workspace paths. Co-authored-by: Cursor <cursoragent@cursor.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ROCm/rocm-systemsdevelopsource. Theprojects/rocprofiler-computeapproval is already in.github/skill_owners.json.developatf2b844211d55. Catalog names match the upstream folders, so noasaliases are used.rocprof-compute-pc-samplingprofiles withrocprof-compute.rocprofiler-pc-samplingdrivesrocprofv3directly. Both can be vendored; they should not be treated as the same skill when a routing room contains both.Readiness follow-up
Linux/MI300 runner metadata, explicit ROCm prerequisites, the gfx950 memory-analysis links, the rocprofv3 routing boundary, and structural Skillscope CI are in ROCm/rocm-systems#14099. That pull request needs review before it can reach
develop. After it merges, the nightly federation bump brings those files into this catalog. This machine has no supported accelerator (rocprof-compute --specsreports no supported arch), so behavioral grading was not run here.Test plan
uv run .github/scripts/federate_skills.py --check-catalog./.github/scripts/check.shdevelopcommitrocprofiler-pc-samplingevals/machine.ymllands from the upstream pull requestMade with Cursor