Skip to content

Add adapter session tests - #3522

Merged
tiagoshibata merged 10 commits into
masterfrom
user/ticastro/adapter-tests
Apr 24, 2020
Merged

Add adapter session tests#3522
tiagoshibata merged 10 commits into
masterfrom
user/ticastro/adapter-tests

Conversation

@tiagoshibata

Copy link
Copy Markdown
Contributor

Description: Test WinML adapter session.

@tiagoshibata
tiagoshibata requested a review from a team as a code owner April 14, 2020 17:43
Comment thread winml/test/adapter/AdapterSessionTest.cpp Outdated
#include "core/session/ort_env.h"

using namespace winrt;
using namespace winrt::Windows::AI::MachineLearning;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am removing windows. in my pr

Comment thread winml/test/adapter/AdapterSessionTest.cpp
Comment thread winml/test/adapter/AdapterSessionTest.cpp Outdated
Comment thread winml/test/adapter/AdapterSessionTest.cpp Outdated
@tiagoshibata tiagoshibata changed the title [WIP] Add adapter session tests Add adapter session tests Apr 15, 2020
@tiagoshibata
tiagoshibata force-pushed the user/ticastro/adapter-tests branch 2 times, most recently from a81af7e to 3ae8b20 Compare April 24, 2020 17:49
#include "core/session/ort_env.h"

using namespace _winml;
using namespace winrt::Windows::Foundation::Collections;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use the aliases here to "future proof"

wfc;
wgu;
wm;
ws;
wss

@tiagoshibata
tiagoshibata merged commit f48b9e2 into master Apr 24, 2020
@tiagoshibata
tiagoshibata deleted the user/ticastro/adapter-tests branch April 24, 2020 21:39
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