-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
test(psram): Add PSRAM test #10409
test(psram): Add PSRAM test #10409
Conversation
👋 Hello lucasssvaz, we appreciate your contribution to this project! Click to see more instructions ...
Review and merge process you can expect ...
|
Test Results 56 files 56 suites 4m 18s ⏱️ Results for commit c9b433a. ♻️ This comment has been updated with latest results. |
f106acf
to
9d9e8dd
Compare
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
f409f32
to
8799a3c
Compare
@P-R-O-C-H-Y PTAL |
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.
LGTM. @lucasssvaz I am worried about the S3 as there are 2 PSRAM options (OPI and quad). Do we have both runners available and also how the test recognize which to chose?
Yeah, forgot to consider this. I need to check which PSRAM type our hardware runners have. If they were running properly that would be easier XD.
|
Description of Change
Add validation test for PSRAM
Tests scenarios
CI and Locally