Skip to content
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

Bg/fix bn solvers is applicable #3355

Merged

Conversation

bghimireamd
Copy link
Contributor

@bghimireamd bghimireamd commented Nov 2, 2024

Fix Gtest for ock and ck kernel for batch norm
fix driver cmd

src/fusion.cpp Outdated Show resolved Hide resolved
@BrianHarrisonAMD BrianHarrisonAMD merged commit 75f15a6 into bharriso/bn-ocl-bf16 Nov 2, 2024
3 of 122 checks passed
@BrianHarrisonAMD BrianHarrisonAMD deleted the bg/fix_bn_solvers_is_applicable branch November 2, 2024 20:00
BrianHarrisonAMD added a commit that referenced this pull request Nov 2, 2024
* Add bfloat16 support to OCL bn kernels

* Fix issue

* Fix spacing

* Add bf16 support for bwd OCL spatial

* Further changes of OCL kernels for bfp16

* Add  bf16 support for remaining OCL solvers

* Fix compilation error

* Fix compilation errors

* Fix formatting

* Fix extra bracket being added

* Fix kernel compilation and issue with variant selection for BF16 mixed

* Swap solver order to prefer OCL over CK by default

* Update batchnorm bfp16mix to match fp16mix declaration

* Fix improper cast

* Fix formatting

* Fix kernel typo for activ

* Bg/fix bn solvers is applicable (#3355)

* fix driver for old ocl kernel

* fix template name

* fix review comments

* fix isApplicable for CK and OCL bn spatial

* fix gtest issue in bn for ck and ocl

* fix minor issue

* fix clang format

* hip tidy

* fix gtest sample and fix hiptidy

* * move type check logic to problem description of batch norm
* add type checks to other ocl solvers
* fix other minor issues

* fix network cache

* fix minor testing issue

* disable ck bn for now

---------

Co-authored-by: BrianHarrisonAMD <[email protected]>

* Fix formatting

---------

Co-authored-by: Brad Pepers <[email protected]>
Co-authored-by: bibek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants