-
Notifications
You must be signed in to change notification settings - Fork 15
Adding power profiles to our MEP E2E codebase. #74
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
|
@AnkitaWSE please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
aastitva
left a comment
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.
There are few places where there is no functional changes, it's just space added. Please remove those spaces
Removed new pixels as per Tapas's comments If addition of new pixels is the functionality of current feature you implemented. If it is not then going forward always raise a separate PR for different functionalities.
| $scenarioLogFolder = "CameraAppTest\$camsnario\$vdoResDetails\$ptoResDetails\$devPowStat\$VFdetails\$toggleEachAiEffect" | ||
| $powerProfile = $powerProfile -replace ' ', '' | ||
| # Updated scenario folder path to include power profile | ||
| $scenarioLogFolder = "CameraAppTest\$powerProfile\$camsnario\$vdoResDetails\$ptoResDetails\$devPowStat\$VFdetails\$toggleEachAiEffect" |
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.
@AnkitaWSE Can you please add $powerProfile after $devPowStat
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.
@AnkitaWSE Can we add an option to run the test with just default or any specific power mode if user doesn't want to run the test for all available Power profile mode.
Can you share the report/log file for the run with these changes.
What changed? - Added Power profiles - Balanced, Best Performance, Best Efficiency.
Why changed? - To test the E2E package with all these power profiles.
How did you test the change? By running ReleaseTest.ps1.
Related Issues (if any):