-
Notifications
You must be signed in to change notification settings - Fork 242
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
[tests] Added smoke tests for conv solvers (ConvAsmImplicitGemmV4R1DynamicFwd_1x1 and more). Some fixes. #1911
base: develop
Are you sure you want to change the base?
Conversation
…icitGemmGTCDynamicWrwXdlopsNHWC.
…plicitGemmGTCDynamic(Fwd|Bwd)Xdlops
…h results in omission of all custom tests
…d() check from ConvAsm1x1UV2 is it does not support FP16
…ltipassWrW (3-3 only)
…t potential performance degradation after tests.
…mmGTCDynamicFwdXdlopsNHWC
…mmGTCDynamicBwdXdlopsNHWC
…mmGTCDynamicWrwXdlopsNHWC
…mmGTCDynamicFwdDlopsNCHWC
# RESOLVED Conflicts: # test/CMakeLists.txt
…wdXdlops for MI200
@averinevg @shurale-nkn @johnny-keker @junliume Solver tests updated for MI300. Ready for review & CI testing. |
@atamazov another round of CI has just started. |
I think fresh develop needs to be merged once more. |
@junliume @johnny-keker Please start another CI round, thanks! |
@junliume @johnny-keker Can you please send me failing logs? Please also let me know which docker image is currently used on CI. Thanks! |
Hi @atamazov it failed in
|
@junliume Thanks! Which docker image is currently used on CI? |
RESOLVED Conflicts: - test/CMakeLists.txt
…onvHipImplicitGemmBwdXdlops for MI200 && FP32
@junliume wrt #1911 (comment):
Merged from the recent develop as well. Can you please start CI again. Thanks! |
@@ -237,6 +238,9 @@ option( WORKAROUND_ISSUE_1187 "" ${WORKAROUND_ISSUE_1187_DEFAULT}) | |||
set_var_to_condition(WORKAROUND_ISSUE_1148_DEFAULT (MIOPEN_TEST_GFX103X OR MIOPEN_TEST_GFX110X) AND MIOPEN_TEST_FLOAT) | |||
option( WORKAROUND_ISSUE_1148 "" ${WORKAROUND_ISSUE_1148_DEFAULT}) | |||
|
|||
set_var_to_condition(WORKAROUND_ISSUE_2173_DEFAULT (MIOPEN_TEST_GFX90A OR MIOPEN_TEST_GFX908 OR MIOPEN_TEST_GFX94X) AND MIOPEN_TEST_FLOAT) |
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.
TODO: Enable W/A for Debug builds only
@atamazov could we revisit this PR? Update it and I can kick off a whole set of CI on it. |
@junliume Sure, as soon as I have time. Quite a lot of work required. |
This is the last PR in series that aims to cover all convolution Solvers with smoke tests.
@junliume Proposed reviewers: @averinevg (mostly to share knowledge), @shurale-nkn. https://github.com/ROCmSoftwarePlatform/MIOpen/labels/testing https://github.com/ROCmSoftwarePlatform/MIOpen/labels/urgency_high
This PR adds smoke tests for conv solvers:
WORKAROUND_ISSUE_2173
introduced.By products:
MIOPEN_FIND_ENFORCE=SEARCH_DB_UPDATE MIOPEN_DEBUG_TUNING_ITERATIONS_MAX=5
->${ENFORCE_TUNING_5}
TODO
GFX110X_ENABLED
where necessary (minimum)GFX110X_ENABLED
for solvers enabled in [GFX11] Add support for gfx1101 target ID #1947ConvHipImplicitGemmBwdXdlops
from [CK API] 2d backward data convolution composable kernel integration #1874GFX110X_ENABLED
for solvers listed in (2) at Deprecate some old static solvers for new ASICs #2080GFX94X_ENABLED
andMIOPEN_DEBUG_ENABLE_DEPRECATED_SOLVERS=1
where necessary, see Deprecate some old static solvers for new ASICs #2080