-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Assert less than and equal status check #1339
Conversation
Codecov ReportBase: 71.41% // Head: 71.05% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## devel #1339 +/- ##
==========================================
- Coverage 71.41% 71.05% -0.36%
==========================================
Files 57 57
Lines 6093 6125 +32
Branches 1081 1087 +6
==========================================
+ Hits 4351 4352 +1
- Misses 1740 1771 +31
Partials 2 2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
buildspecs: | ||
assert_le_example: | ||
type: script | ||
executor: generic.local.bash | ||
description: Run stream test with metrics example using assert less than equal | ||
env: | ||
OMP_NUM_THREADS: 4 | ||
run: | | ||
wget https://raw.githubusercontent.com/jeffhammond/STREAM/master/stream.c | ||
gcc -openmp -o stream stream.c | ||
./stream | ||
metrics: | ||
copy: | ||
type: float | ||
regex: | ||
exp: 'Copy:\s+(\S+)\s+.*' | ||
stream: stdout | ||
item: 1 | ||
scale: | ||
type: float | ||
regex: | ||
exp: 'Scale:\s+(\S+)\s+.*' | ||
stream: stdout | ||
item: 1 | ||
add: | ||
type: float | ||
regex: | ||
exp: 'Add:\s+(\S+)\s+.*' | ||
stream: stdout | ||
item: 1 | ||
triad: | ||
type: float | ||
regex: | ||
exp: 'Triad:\s+(\S+)\s+.*' | ||
stream: stdout | ||
item: 1 | ||
status: | ||
assert_le: | ||
- name: copy | ||
ref: 5000 | ||
- name: scale | ||
ref: 5500 | ||
- name: add | ||
ref: 6000 | ||
- name: triad | ||
ref: 6500 |
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.
i think for now let's keep these files but i think we may want to merge this one test into one single test such that we can have all the checks in one test
@Xiangs18 thanks this PR looks good. I will merge this in |
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.
Hi @shahzebsiddiqui, please take a look at MR and provide feedback. Thank you!