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

[Segment Anything][SAM ViT-B Decoder] Failed to execute 'constant' on 'MLGraphBuilder': This MLGraphBuilder has already built a graph #20

Closed
JiangYueBook opened this issue Jul 26, 2024 · 4 comments · Fixed by #21

Comments

@JiangYueBook
Copy link

Repro Steps:

  1. Launch latest Chrome Canary on Windows 11
    C:\Users\AppData\Local\Google\Chrome SxS\Application\chrome.exe" --enable-features=WebMachineLearningNeuralNetwork

  2. Visit https://microsoft.github.io/webnn-developer-preview/demos/segment-anything/

  3. Check demo loading logs

Actual Result:

   [Session Create] SAM ViT-B Decoder (FP16) failed, Failed to execute 'constant' on 'MLGraphBuilder': This MLGraphBuilder has already built a graph.

Environment:

Chrome Canary: 129.0.6617.0

Device OS CPU GPU
Desktop Windows 11 23H2 i7-12700 Nvidia 2080Ti
@ibelem
Copy link
Contributor

ibelem commented Jul 26, 2024

Thanks for the report @JiangYueBook

  • Canary 129.0.6615.0 PASS
  • Canary 129.0.6617.0 FAIL / Failed to execute 'constant' on 'MLGraphBuilder': This MLGraphBuilder has already built a graph

This issue caused by Allow an MLGraphBuilder to build at most one MLGraph, @Honry will update WebNN EP in ONNX Runtime Web and I will update the ORT Web dists later in demo code.

@fdwr @Adele101

@fdwr
Copy link
Collaborator

fdwr commented Jul 26, 2024

@a-sully Oops, I guess there was some impact. I figured your change would be impact-free for the samples.
@Honry Thank you for updating ORT. I guess the WebNN EP builds multiple partitions within the same graph using the same graphBuilder?

@Honry
Copy link
Contributor

Honry commented Jul 26, 2024

@Honry Thank you for updating ORT. I guess the WebNN EP builds multiple partitions within the same graph using the same graphBuilder?

Yes, I am fixing this.

@Honry
Copy link
Contributor

Honry commented Jul 26, 2024

@Honry Thank you for updating ORT. I guess the WebNN EP builds multiple partitions within the same graph using the same graphBuilder?

Yes, I am fixing this.

PR submitted, @fdwr, PTAL, thanks!

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 a pull request may close this issue.

4 participants