Skip to content

Implement new spec api#9

Merged
huningxin merged 2 commits intowebmachinelearning:mainfrom
fujunwei:new_api
Apr 26, 2021
Merged

Implement new spec api#9
huningxin merged 2 commits intowebmachinelearning:mainfrom
fujunwei:new_api

Conversation

@fujunwei
Copy link
Collaborator

@fujunwei fujunwei commented Apr 15, 2021

  • Rename ModelBuilder to MLGraphBuilder, Model to MLGraph
  • Remove old api Compilation and merge the compute capability to MLGraph
  • Create MLGraph with MLContext and remove the concrete GraphBuilder from backend

endtoend test and lenet example can be passed on both DML and openvino backend.

This pull request is to fix #7, @huningxin PTAL, thanks.

@fujunwei
Copy link
Collaborator Author

@huningxin please take a look again, thanks.

@fujunwei fujunwei force-pushed the new_api branch 2 times, most recently from dc37b90 to 7de99fd Compare April 25, 2021 06:25
Copy link
Collaborator

@huningxin huningxin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @fujunwei !

@huningxin huningxin merged commit b3b0548 into webmachinelearning:main Apr 26, 2021
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.

Implement the MLContext, MLGraphBuilder and MLGraph

2 participants