Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Implement the gst_encoder_psnr scenario (New) #1491

Merged
merged 14 commits into from
Oct 15, 2024
Merged

Implement the gst_encoder_psnr scenario (New) #1491

merged 14 commits into from
Oct 15, 2024

Conversation

baconYao
Copy link
Contributor

@baconYao baconYao commented Sep 20, 2024 •
edited
Loading

Description

This PR aims to implement the gst_encoder_PSNR scenario

Resolved issues

[PEB-686] [checkbox] Implement gst_encoder_PSNR scenario

Documentation

OQ013 - Hardware Video Codec Testing Document

Spec - Rity 24.0 - Encoder (Page 24 - 29)

Tests

Sideload Submissions - Pass

Sideload Submissions - Fail

@baconYao baconYao added the ce-oem The PR only contains to the ce-oem provider under the contrib area label Sep 20, 2024
@baconYao baconYao changed the title Init version of gst_encoder_psn scenario Implement the gst_encoder_psn scenario (New) Sep 23, 2024
@baconYao baconYao requested a review from a team September 23, 2024 08:46
@baconYao baconYao marked this pull request as ready for review September 24, 2024 01:40
@baconYao baconYao changed the title Implement the gst_encoder_psn scenario (New) Implement the gst_encoder_psnr scenario (New) Sep 24, 2024
tomli380576
tomli380576 approved these changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@tomli380576 tomli380576 left a comment

Choose a reason for hiding this comment

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

lgtm

stanley31huang
stanley31huang approved these changes Oct 15, 2024
View reviewed changes
Copy link
Collaborator

@stanley31huang stanley31huang left a comment

Choose a reason for hiding this comment

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

LGTM

@baconYao baconYao merged commit f67ed8b into main Oct 15, 2024
12 checks passed
@baconYao baconYao deleted the gst_encodec_psnr branch October 15, 2024 02:23
@codecov Codecov
Copy link

codecov bot commented Oct 15, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.09%. Comparing base (bf03f1a) to head (4284397).
Report is 130 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1491   +/-   ##
=======================================
  Coverage   47.09%   47.09%           
=======================================
  Files         368      368           
  Lines       39435    39435           
  Branches     6667     6667           
=======================================
  Hits        18573    18573           
  Misses      20156    20156           
  Partials      706      706

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stanley31huang stanley31huang stanley31huang approved these changes

@tomli380576 tomli380576 tomli380576 approved these changes

Assignees

@tomli380576 tomli380576

Labels
ce-oem The PR only contains to the ce-oem provider under the contrib area
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@baconYao @tomli380576 @stanley31huang