-
Notifications
You must be signed in to change notification settings - Fork 63
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
Fix for issue #395 #414
Fix for issue #395 #414
Conversation
@henryleberre can confirm but I think this might be backwards. lhs -> master and rhs -> PR. If the variable If the variable At least that's my read of the code... Code: Lines 135 to 138 in f23fceb
MFC/.github/workflows/bench.yml Line 38 in f23fceb
Lines 108 to 122 in 64d9677
and your code says:
|
Let's consider an example: $ ./mfc.sh bench_diff lhs rhs where Say that for a given example case, you have:
So, As a sidenote, there are a lot of cons.write(f"""\
Line 1
Line 2
""") |
There should just be a line of text above the results that says "numbers < 1 indicate the PR is faster than master, > 1 indicate PR is slower" (if that is actually the case) |
PR merges are on hold until benchmark CI is working (issue #419) |
The past few tests have either failed or have the wrong output so I'm just making sure that's not the issue. |
@okBrian please sync your branch with master, which I just merged to (that's why your current CI tests are failing) |
CI benchmarking is failing and it seems related to this PR |
This might need another merge with master |
Description
Fixed the clarity issue for the CI benchmark comparison message when comparing a PR and master.
Example:
Fixes #395
Type of change
Scope
How Has This Been Tested?
Not sure how to run the diff benchmark comparison without the github runner
Test Configuration:
Compiled on a
Ubuntu 22.04.4 using Docker
Intel i7-1165G7
Checklist
docs/
)examples/
that demonstrate my new feature performing as expected.They run to completion and demonstrate "interesting physics"
./mfc.sh format
before committing my codeIf your code changes any code source files (anything in
src/simulation
)To make sure the code is performing as expected on GPU devices, I have:
nvtx
ranges so that they can be identified in profiles./mfc.sh run XXXX --gpu -t simulation --nsys
, and have attached the output file (.nsys-rep
) and plain text results to this PR./mfc.sh run XXXX --gpu -t simulation --omniperf
, and have attached the output file and plain text results to this PR.