Skip to content

Camera snapshot enhancements and helper function improvements for gst#483

Merged
smuppand merged 2 commits into
qualcomm-linux:mainfrom
nitinn22:feature/camera-tests-enhancements
Jun 9, 2026
Merged

Camera snapshot enhancements and helper function improvements for gst#483
smuppand merged 2 commits into
qualcomm-linux:mainfrom
nitinn22:feature/camera-tests-enhancements

Conversation

@nitinn22

@nitinn22 nitinn22 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Added helper enhancements in lib_gstreamer.sh
  • Improved camera test execution logic in run.sh (snapshot handling)

Changes

  1. gstreamer utilities refactoring and enhancements
  2. Camera snapshot test execution improvements

Testing

  • Verified camera test execution with updated run.sh
  • Validated helper functions for pipeline handling
    Testing performed locally on device as with LAVA framework we had issues with original PR as well.

nitinn22 added 2 commits June 7, 2026 19:24
Signed-off-by: nitinn <nitinn@qti.qualcomm.com>
Signed-off-by: nitinn <nitinn@qti.qualcomm.com>

@smuppand smuppand left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@nitinn22 Please squash the 2 commits

@nitinn22

nitinn22 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

@smuppand , it was intentional to keep 2 commits, one for helper and one for runner as per discussion in past PRs.

@smuppand

smuppand commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

@smuppand , it was intentional to keep 2 commits, one for helper and one for runner as per discussion in past PRs.

Thanks for clarifying. Since the split is intentional and scoped as:

  1. lib_gstreamer.sh helper changes
  2. Camera_Tests/run.sh runner changes

@smuppand smuppand left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@smuppand smuppand merged commit a510bf9 into qualcomm-linux:main Jun 9, 2026
12 checks passed
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.

2 participants