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.

Sign up for GitHub

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

Jump to bottom

CopyTensor in c-api #31

Closed
ashmanov opened this issue Feb 15, 2018 · 2 comments
Closed

CopyTensor in c-api #31

ashmanov opened this issue Feb 15, 2018 · 2 comments

Comments

@ashmanov
Copy link

Hello! Why not wrap CopyTensor from C++ api as miopenTransformTensor in c-api?
@ashmanov
Copy link
Author

ashmanov commented Mar 4, 2018

It looks like it is easy to add it and developers just forgot to add it to C-API.

@daniellowell
Copy link
Contributor

As I understand it, transfer tensor isn't a simple copy over, but changing the data layout. In that case wrapping copyTensor would not be sufficient. Also, MIOpen currently only supports NCHW as our 4D tensor layout.

pfultz2 pushed a commit that referenced this issue Mar 30, 2018
@ce1adon
Merge pull request #31 from AMDComputeLibraries/develop
4d3a27a 
update to develop
@alexandraBara alexandraBara mentioned this issue Sep 11, 2020
Closed
cderb added a commit that referenced this issue Oct 6, 2021
@cderb
Squashed 'fin/' changes from 2deafda81..09936eef3
e27c3d2 
09936eef3 clean up objects
9fdc31638 Merge remote-tracking branch 'origin/develop' into cderb_pdb_verif
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)

git-subtree-dir: fin
git-subtree-split: 09936eef3ee6956ce41951cb196cb63223204477
ltqin pushed a commit that referenced this issue Oct 28, 2021
Squashed 'src/composable_kernel/' changes from 31b4035..646fcc268
781bde2 
646fcc268 Merge pull request #47 from ROCmSoftwarePlatform/develop
6014185 [Bug Fix] GridwiseGemm_bk0mk1_bk0nk1_mn_xdlops_v2r4 loop issue (#44)
3e91137 Merge pull request #46 from ROCmSoftwarePlatform/miopen_downstream_all
211dae8 Merge branch 'develop' into miopen_downstream_all
5890e30 [Composable Kernel] update develop branch code to ck_upstream
d5297ab fix bug in gridwise gemm xdlops v2r3 (#45)
38a90b6ed Merge pull request #43 from ROCmSoftwarePlatform/develop
c301879 bug fix (#39)
fd49ff8 add nchw atomic , nhwc and nhwc atomic method   for backward weight (#30)
b2dc55f [MIOpen Downstream] Fix Reduction Kernel (#34)
b3e8d57 Tweak GEMM kernel (#38)
846f462 Add VectorType support into StaticBuffer (#27)
dfb80c4 [Enhancements] Several bugfixes and refactoring of dynamic generic reduction  (#1156)
8557901 Merge pull request #1165 from ROCmSoftwarePlatform/develop
f305beb Merge pull request #31 from ROCmSoftwarePlatform/miopen_downstream-dynamic_reduction_pr
b725e3f Merge remote-tracking branch 'origin/develop' into miopen_downstream-dynamic_reduction_pr
88833bd9a Merge pull request #32 from ROCmSoftwarePlatform/develop
df0d681 :Merge remote-tracking branch 'origin/develop' into CK_upstream
f3acd25 Add  a version of Merge transform that use integerdivision and mod (#25)
1961390 GEMM driver and kernel (#29)
627d8ef Backward weight v4r4r2 with xdlops (#18)
10bb811 Misc fixes (#24)
9e80cdc [SWDEV-281541][MSRCHA-100] Implementation of Dynamic Generic Reduction  (#1108)
a7a758d GlobalAtomicAdd for fp32/int32 (#23)
9d3f634 Xdlops refactor fix (#22)
c6f26bb magic division use __umulhi() (#19)
6fe3627 Composable kernel init integration v3 (#1097)
a2ad6d3 refactor dynamic xdlops iGemm (#13)
ba6f79a Added host_conv_wrw for verification (#15)

git-subtree-dir: src/composable_kernel
git-subtree-split: 646fcc268ede841a16cdaafb68aa64803d8390e1
cderb added a commit that referenced this issue Nov 17, 2021
@cderb
Squashed 'fin/' changes from 53d2563fe..f3f7fed18
f86de63 
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: f3f7fed18ba0473c6ff0f388ff2d0deffde01a46
alexandraBara added a commit that referenced this issue Jan 14, 2022
@alexandraBara
Squashed 'fin/' changes from 53d2563fe..0e164bf66
0bb7322 
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: 0e164bf66280932586e25e15d09c60b02cbd6fc3
junliume pushed a commit that referenced this issue Jan 19, 2022
@alexandraBara
[TUNA] FIN subtree updates JSON versioning (#1376)
ac6281a 
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: 0e164bf66280932586e25e15d09c60b02cbd6fc3
cderb added a commit that referenced this issue Mar 8, 2022
@cderb
Squashed 'fin/' changes from 53d2563fe..a30a51bc6
3844958 
a30a51bc6 remove unused header
7d2fd834c reduce scope of variable
f6e9abe79 clang format
834e9a397 remove comment
c8d6eb1a0 workspace rename
aa7d2ea24 Merge remote-tracking branch 'origin/develop' into cderb/miopen_perf
aaf13fb12 add to print for debug
ebd9aa6bd update member name (#43)
34e11fa70 Merge remote-tracking branch 'origin/develop' into cderb/miopen_perf
cb6c19d13 add search+update directives to execution context, add json examples for perf eval
85029077b connecting new fin functions for perf eval
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
4d1e031fd add outputs and definitions
952538cb8 adding perf eval function, in progress
0e164bf66 setting json version (#37)
617dccd9c rename
5c35ae886 fixes for collecting kernel blobs
5cfea7c43 syntax fixes
2f2a4ed9f add test file
7175019f5 first rendition of perf_compile
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: a30a51bc6b4e3b1dedf6a37b429b287abf5784fc
cderb added a commit that referenced this issue Aug 3, 2022
@cderb
Squashed 'fin/' changes from 53d2563fe..30d699b9e
41b34da 
30d699b9e Perf Eval Update (#60)
3535b948c PerfCompile and PerfEval changes (#59)
de79468d2 remove unneccessary solution check, add check for previously modified kernel names (#56)
6924286a2 miopen hash update (#55)
530399575 Refactor googletest infra to align with MIOpen (#53)
71c50d146 Datatype fix for BN (#57)
8abe2f5c6 Perf Eval updates, Add find info (#51)
e1c1ef0f5 filter find compile by solver input (#54)
722feea66 sp/chk precomp kernel 264 (#41)
b9aba2034 Batch norm find compile (#50)
359f3da80 Fix missing link directives in fin binary (#48)
a4020c1ba Cache Miss Fixes (#46)
2ec7ef44d Enable google test and compiling fin in the CI (#47)
8b6b453bc Applicability support for batch norm (#45)
44323aae9 Perf compile/eval for fin (#42)
ebd9aa6bd update member name (#43)
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: 30d699b9edc014c6076a9649f849bd3c4588d4ab
averinevg pushed a commit that referenced this issue Aug 19, 2022
@cderb @junliume
[Enhancement][Fin] Enhance test for pdb records (#1669)
84d4399 
* add perf cfg validity test to TestSysDbRecord

* remove debug prints

* removing invalid entries from all perf dbs

* VACUUM sqlite

* Squashed 'fin/' changes from 53d2563fe..30d699b9e

30d699b9e Perf Eval Update (#60)
3535b948c PerfCompile and PerfEval changes (#59)
de79468d2 remove unneccessary solution check, add check for previously modified kernel names (#56)
6924286a2 miopen hash update (#55)
530399575 Refactor googletest infra to align with MIOpen (#53)
71c50d146 Datatype fix for BN (#57)
8abe2f5c6 Perf Eval updates, Add find info (#51)
e1c1ef0f5 filter find compile by solver input (#54)
722feea66 sp/chk precomp kernel 264 (#41)
b9aba2034 Batch norm find compile (#50)
359f3da80 Fix missing link directives in fin binary (#48)
a4020c1ba Cache Miss Fixes (#46)
2ec7ef44d Enable google test and compiling fin in the CI (#47)
8b6b453bc Applicability support for batch norm (#45)
44323aae9 Perf compile/eval for fin (#42)
ebd9aa6bd update member name (#43)
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: 30d699b9edc014c6076a9649f849bd3c4588d4ab

* Squashed 'fin/' changes from 30d699b9e..ea5c844af

ea5c844af fix direction test
3aa412ee1 Update to use revised testSysDbRecord miopen function

git-subtree-dir: fin
git-subtree-split: ea5c844aff8b5d46537aa59034a596fd15cd9e1e

* rename pipe step

* Squashed 'fin/' changes from ea5c844af..c702cb968

c702cb968 format

git-subtree-dir: fin
git-subtree-split: c702cb96800a03b17ee17d03a015dfa38e3883b9

* Squashed 'fin/' changes from c702cb968..d5397abd3

d5397abd3 rename targets

git-subtree-dir: fin
git-subtree-split: d5397abd37b6908bcd96ef750ea5a3ace04cdf3c

* rename archive

Co-authored-by: Jun Liu <Liu.Jun@amd.com>
cderb added a commit that referenced this issue Oct 5, 2022
@cderb
Squashed 'fin/' changes from 53d2563fe..e05dcb421
8875272 
e05dcb421 perf db validation fix (#68)
260d9465d Add INT8 as a data_type v2 (#67)
b6a5b2a77 sync with fin folder in miopen (#62)
0e03399ec prep for Palamida scan (#63)
e6bd05c33 Performance db testing (#61)
30d699b9e Perf Eval Update (#60)
3535b948c PerfCompile and PerfEval changes (#59)
de79468d2 remove unneccessary solution check, add check for previously modified kernel names (#56)
6924286a2 miopen hash update (#55)
530399575 Refactor googletest infra to align with MIOpen (#53)
71c50d146 Datatype fix for BN (#57)
8abe2f5c6 Perf Eval updates, Add find info (#51)
e1c1ef0f5 filter find compile by solver input (#54)
722feea66 sp/chk precomp kernel 264 (#41)
b9aba2034 Batch norm find compile (#50)
359f3da80 Fix missing link directives in fin binary (#48)
a4020c1ba Cache Miss Fixes (#46)
2ec7ef44d Enable google test and compiling fin in the CI (#47)
8b6b453bc Applicability support for batch norm (#45)
44323aae9 Perf compile/eval for fin (#42)
ebd9aa6bd update member name (#43)
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: e05dcb42187f05fe0d0d1b05b822dc4b750f199e
junliume added a commit that referenced this issue Oct 6, 2022
@cderb @junliume
Perf DB cleanup + Fin pdb verification fix (#1810)
e7d3c60 
* remove datatype 0,1 from perf_db

* rm invalid fp16 entries from pdb

* Squashed 'fin/' changes from 53d2563fe..e05dcb421

e05dcb421 perf db validation fix (#68)
260d9465d Add INT8 as a data_type v2 (#67)
b6a5b2a77 sync with fin folder in miopen (#62)
0e03399ec prep for Palamida scan (#63)
e6bd05c33 Performance db testing (#61)
30d699b9e Perf Eval Update (#60)
3535b948c PerfCompile and PerfEval changes (#59)
de79468d2 remove unneccessary solution check, add check for previously modified kernel names (#56)
6924286a2 miopen hash update (#55)
530399575 Refactor googletest infra to align with MIOpen (#53)
71c50d146 Datatype fix for BN (#57)
8abe2f5c6 Perf Eval updates, Add find info (#51)
e1c1ef0f5 filter find compile by solver input (#54)
722feea66 sp/chk precomp kernel 264 (#41)
b9aba2034 Batch norm find compile (#50)
359f3da80 Fix missing link directives in fin binary (#48)
a4020c1ba Cache Miss Fixes (#46)
2ec7ef44d Enable google test and compiling fin in the CI (#47)
8b6b453bc Applicability support for batch norm (#45)
44323aae9 Perf compile/eval for fin (#42)
ebd9aa6bd update member name (#43)
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: e05dcb42187f05fe0d0d1b05b822dc4b750f199e

* fix clang-format issue

Co-authored-by: Jun Liu <Liu.Jun@amd.com>
cderb added a commit that referenced this issue Nov 21, 2022
@cderb
Squashed 'fin/' changes from 53d2563fe..49e3e3a62
4042531 
49e3e3a62 clang format
db80b1777 update to using TestPerfCfgParams for pdb validity checks
e48a4fd3a format
a4f85842c exception for non-tunable solvers in params check
d58c42bbd Check params at end of perf tuning (#70)
1a3b47c7b Return status for failed compile commands (#69)
d59962752 out_layout -> in_layout
6ba7a8f3f Rename conv_mode to mode (#64)
513a3da1b [bg/LWPTUNA-173] (#65)
e05dcb421 perf db validation fix (#68)
260d9465d Add INT8 as a data_type v2 (#67)
b6a5b2a77 sync with fin folder in miopen (#62)
0e03399ec prep for Palamida scan (#63)
e6bd05c33 Performance db testing (#61)
30d699b9e Perf Eval Update (#60)
3535b948c PerfCompile and PerfEval changes (#59)
de79468d2 remove unneccessary solution check, add check for previously modified kernel names (#56)
6924286a2 miopen hash update (#55)
530399575 Refactor googletest infra to align with MIOpen (#53)
71c50d146 Datatype fix for BN (#57)
8abe2f5c6 Perf Eval updates, Add find info (#51)
e1c1ef0f5 filter find compile by solver input (#54)
722feea66 sp/chk precomp kernel 264 (#41)
b9aba2034 Batch norm find compile (#50)
359f3da80 Fix missing link directives in fin binary (#48)
a4020c1ba Cache Miss Fixes (#46)
2ec7ef44d Enable google test and compiling fin in the CI (#47)
8b6b453bc Applicability support for batch norm (#45)
44323aae9 Perf compile/eval for fin (#42)
ebd9aa6bd update member name (#43)
d6d798efe add cu count (#39)
8e1989a9f Add find option for selecting only dynamic solvers (#38)
0e164bf66 setting json version (#37)
f3f7fed18 Remove function redefinition (#36)
e1de51a58 Performance DB de-serialize test (#34)
043cdcdaa Layout support in Fin (#33)
3a1d58236 Hotfix (#32)
ee3f0d543 4.4 Tuning Bugfixes (#31)
832dbe234 Tunability Reporting (#27)
a564a229f include gfx90a_110 (#28)

git-subtree-dir: fin
git-subtree-split: 49e3e3a62a7cc54adacbeea95680d35f9a4685de
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ashmanov @daniellowell