-
Notifications
You must be signed in to change notification settings - Fork 21
Add FastCV OpenCV SFM validation test script #196
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
This script performs rootfs resizing, sets permissions for OpenCV binaries, installs required IPK packages, and runs the OpenCV SFM test suite with performance parameters. It validates the output to ensure all 19 tests pass. Signed-off-by: Anil Yadav <[email protected]>
Please ensure that script has required execute permissions |
# shellcheck disable=SC1090,SC1091 | ||
. "$TOOLS/functestlib.sh" | ||
|
||
TESTNAME="fastCV" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to FastCV as dir structure is different
|
||
# Step 3: Install IPK packages | ||
log_info "Installing IPK packages from /usr/bin/FastCV/opencv..." | ||
cd /usr/bin/FastCV/opencv || exit 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add some debug logs before exit so that it will be easy for debugging
|
||
# Step 1: Resize rootfs | ||
log_info "Resizing rootfs partition..." | ||
resize2fs /dev/disk/by-partlabel/rootfs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use available functions
qcom-linux-testkit/Runner/utils/functestlib.sh
Line 3399 in 59365c8
ensure_rootfs_min_size() { |
# Step 2: Set permissions | ||
log_info "Setting permissions for OpenCV binaries..." | ||
chmod 777 /usr/bin/FastCV/opencv | ||
chmod 777 /usr/bin/opencv* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to review the current execute permissions first, rather than immediately assigning 777 without checking.
# Step 3: Install IPK packages | ||
log_info "Installing IPK packages from /usr/bin/FastCV/opencv..." | ||
cd /usr/bin/FastCV/opencv || exit 1 | ||
opkg install *.ipk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you determine if any ipk failed to install?
Add FastCV OpenCV SFM validation test script
This script performs rootfs resizing, sets permissions for OpenCV binaries,
installs required IPK packages, and runs the OpenCV SFM test suite with
performance parameters. It validates the output to ensure all 19 tests pass.
Signed-off-by: Anil Yadav [email protected]