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

Add some notes for flags to README #300

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mingmingtasd
Copy link
Contributor

Add some notes for --disable_webnn_for_gpu, --disable_webnn_for_npu and --disable_dml_meta_commands_for_gpu in README.
PTAL, thanks!
@ibelem @Honry @huningxin

README.md Outdated
@@ -56,6 +56,11 @@ To get started with WebNN on Intel AI PCs you will need:
#### Running WebNN on CPU or GPU
1. Launch Chrome Canary or Edge Canary

* **Notes**:

1. Running WebNN on Qualcomm GPUs is disabled by default. You can use the `--disable_webnn_for_gpu=0` flag to bypass. However, the functionality is not guaranteed.
Copy link
Contributor

@ibelem ibelem Dec 20, 2024

Choose a reason for hiding this comment

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

I think it's better for Qualcomm to add such notes, not from Intel. @mingmingtasd We don't know if Qualcomm guarantees the functionality or not at all.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I remember we want a README that includes descriptions of all the flags. I can remove the note for
disable_webnn_for_gpu for Qualcomm, then that means if people run on Qualcomm and see GPU blocked issue, it's not our responsibility to tell them: you should use --disable_webnn_for_gpu=0 flag. WDYT?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But agree to remove the sentence "However, the functionality is not guaranteed." for Qualcomm, we should not judge other vendors

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done by the new commit, PTAL, thanks! @ibelem

@@ -84,7 +89,8 @@ At present, the [image classification](https://webmachinelearning.github.io/webn

* **Notes**:

Running WebNN on NPU requires a higher version of DirectML.dll than the one in the Windows system. Using the `--use-redist-dml` flag will allow Google Chrome Canary to load the downloaded DirectML.dll with a sufficiently high version.
1. Running WebNN on NPU requires a higher version of DirectML.dll than the one in the Windows system. Using the `--use-redist-dml` flag will allow Google Chrome Canary to load the downloaded DirectML.dll with a sufficiently high version.
2. Since the NPU drivers may be blocked by default if they are not the latest [Intel® Core™ Ultra NPU Driver on Windows](https://www.intel.com/content/www/us/en/download/794734/intel-npu-driver-windows.html), you can use the `--disable_webnn_for_npu=0` flag to bypass. However, the functionality is not guaranteed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we really need However, the functionality is not guaranteed.? This is known to users by default before officially shipping WebNN.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just intend to be responsible for these new added notes, to notice that if you follow these notes, you may meet functionality issues.

Agree that the However, the functionality is not guaranteed is negative, I am OK to remove them if you think that's really not necessary.

@ibelem
Copy link
Contributor

ibelem commented Jan 2, 2025

@Honry @huningxin PTAL

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