-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[C#] Add missing rocm csharp api #15540
Conversation
@yuslepukhin I can run the tests with Microsoft.ML.OnnxRuntime.EndToEndTests cd csharp/test/Microsoft.ML.OnnxRuntime.EndToEndTests
export SKIPNONPACKAGETESTS=ON
export TESTONGPU=ON
export BUILD_SOURCESDIRECTORY=~/onnxruntime
export PACKAGENAME=Microsoft.ML.OnnxRuntime.ROCm
./runtest.sh ~/onnxruntime/build_rocm_nuget/Release/ 1.15.0-dev-20230418-0937-028d8c4749 The Make this PR ready to trigger all CI pipelines. I'd plan add CI testing integration in one more PR. You can ignore this one for now |
7cf9261
to
5d10892
Compare
bb4e9d4
to
d6966f4
Compare
5d10892
to
1bf8fa5
Compare
d6966f4
to
032704c
Compare
1bf8fa5
to
d892dc7
Compare
050facb
to
b731a52
Compare
d892dc7
to
206ae93
Compare
b731a52
to
81bfb66
Compare
81bfb66
to
95e0e42
Compare
@@ -287,6 +287,11 @@ public struct OrtApi | |||
public IntPtr GetOptionalContainedTypeInfo; | |||
public IntPtr GetResizedStringTensorElementBuffer; | |||
public IntPtr KernelContext_GetAllocator; | |||
public IntPtr GetBuildInfoString; |
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.
backlog: add GetBuildInfoString
stub to avoid rocm apis off-by-one problem. #Resolved
std::string options_str = ""; | ||
|
||
while (it != options.end()) { | ||
if (options_str == "") { |
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.
Consider using:
for (auto const& [key, val] : options) #Resolved
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.
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.
🕐
88def42
to
aa2590d
Compare
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.
Depends on #15535