webmachinelearning / webnn

🧠 Web Neural Network API
https://www.w3.org/TR/webnn/
Other
397 stars 48 forks source link

Broken link #604

Closed xfq closed 8 months ago

xfq commented 8 months ago

The NSNet2 link is 404 in https://github.com/webmachinelearning/webnn/blob/main/explainer.md#key-scenarios .

anssiko commented 8 months ago

It looks like NSNet2-baseline directory was removed in https://github.com/microsoft/DNS-Challenge/commit/d1bcd6927beed89774e50404d4da2fa46b0d8f47

@fdwr or @wchao1115 might know if that model lives somewhere else now or whether it would be appropriate to replace NSNet2 with some other model and update the explainer accordingly.

fdwr commented 8 months ago

🤔 The CR which removed them has no details (random spring cleaning?). I found a few other places that have a version of NSNet2, but they are Python or C-based rather than model files, and they point to the same URL too (now a 404). Also, the Arxiv papers seem related but not specific to it (e.g. https://arxiv.org/abs/2008.06412). So, we could just fix the link point to https://github.com/microsoft/DNS-Challenge/tree/a052ad5a7714bcc6069b515666e837bf973099de/NSNet2-baseline .

anssiko commented 8 months ago

@fdwr thanks for checking, PTAL https://github.com/webmachinelearning/webnn/pull/617 and merge at will.