Skip to content

Publish CVML Playbook#301

Open
danielholanda wants to merge 7 commits into
mainfrom
dholanda/publish_cvml
Open

Publish CVML Playbook#301
danielholanda wants to merge 7 commits into
mainfrom
dholanda/publish_cvml

Conversation

@danielholanda

Copy link
Copy Markdown
Collaborator

Description

This PR publishes the CVML playbook to the official AMD Playbooks website.

@danielholanda danielholanda self-assigned this May 25, 2026
Comment thread playbooks/supplemental/cvml/playbook.json
Comment thread playbooks/supplemental/cvml/playbook.json
Comment thread playbooks/supplemental/cvml/playbook.json Outdated
@sreeram-11 sreeram-11 marked this pull request as ready for review June 1, 2026 20:26
@danielholanda

Copy link
Copy Markdown
Collaborator Author

@adamlam2-amd
Can you take a look?

Comment thread playbooks/supplemental/cvml/playbook.json Outdated
Enabling Linux, Enabling STX and KRK as a supported_platform, tested_platform, and published_platform
@sreeram-11

Copy link
Copy Markdown
Collaborator

@danielholanda,

FYI:

  1. Enabled Linux
  2. Enabled STX and KRK as a supported_platform, tested_platform, and published_platform

@sreeram-11

sreeram-11 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator
  • Installed all dependencies listed in the README on STX and KRK runners (Windows and Linux), effectively setting up these machines to run the playbook.
  • On Linux Halo runners, removed the existing OpenCV installation and reinstalled it using the commands provided in the README.
  • With the updated installation method, OPENCV_INSTALL_ROOT is now set to /home/user/build/install across all Linux runners.
    • CI tests have been updated accordingly to reflect this change.

@sreeram-11

sreeram-11 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

@anna-amd-com. @justichu-amd

NPU driver/runtime seems to be a mandatory requirement, not optional. The current GPU fallback statement is not reliable and should be revisited.

Observed behavior:

  • In all cases, CVML selects:
    • [Face Detector] Using ONNX engine, NPU backend
  • This forces face detection to run on the NPU, with no fallback to GPU.
  • When the NPU driver is not installed, the workflow fails.
image

Conclusion:

  • NPU drivers are effectively required for these environments.

Recommendation:

  • Update the documentation to mark NPU driver installation as mandatory.
  • Remove or correct the GPU fallback statement to reflect actual behavior.

Please confirm and advise.

Comment thread playbooks/supplemental/cvml/README.md

@adamlam2-amd adamlam2-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pending further approval and instruction.

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.

5 participants