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

Scope the faceanalyzer npm package #116

Open
sca1235 opened this issue Aug 19, 2024 · 2 comments
Open

Scope the faceanalyzer npm package #116

sca1235 opened this issue Aug 19, 2024 · 2 comments

Comments

@sca1235
Copy link

sca1235 commented Aug 19, 2024

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.

@sca1235 sca1235 changed the title Scope the npm package Scope the faceanalyzer npm package Aug 19, 2024
@diljale
Copy link
Contributor

diljale commented Aug 20, 2024

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

@sca1235
Copy link
Author

sca1235 commented Oct 1, 2024

Ok thanks hopefully can have this as the tokens expire quickly and auth a lot even though package doesn't update much.

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

No branches or pull requests

2 participants