Azure-Samples / azure-ai-vision-sdk

SDK for Microsoft's Azure AI Vision
MIT License
81 stars 50 forks source link

Scope the faceanalyzer npm package #116

Open sca1235 opened 3 weeks ago

sca1235 commented 3 weeks ago

Please provide us with the following information:

This issue is for a: (mark with an x)

- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Setting the registry=https://pkgs.dev.azure.com/msface/SDK/_packaging/AzureAIVision/npm/registry/ affects the entire project as other packages then all go through that registry.

Is there a way for MS to scope it so we only use that registry for this one package?

https://github.com/Azure-Samples/azure-ai-vision-sdk/blob/c21c5155406c86c5ab254085c0f60f6ccff059d3/samples/web/SetupEnvironment.md

npm seems to only support dynamic registry if you scope it.

we want to use https://registry.npmjs.com/ for all our other packages as it is much faster. When we use the azure registry its too slow.

Any log messages given by the failure

image

Expected/desired behavior

I believe if you supported a scope i.e. similar to how you guys did @azure-rest/ai-vision-image-analysis

we need something like @azure-vision/azure-ai-vision-faceanalyzer to work and not get 404

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful


Thanks! We'll be in touch soon.

diljale commented 3 weeks ago

Thank you for the feedback. We will look into releasing scoped version soon.