-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[metricsadvisor] update min dependency package #25917
Comments
Label prediction was below confidence level |
@wonder6845 what is the issue you're running into? If I'm reading your issue your correctly it seems you're saying MetricsAdvisor and autorest both still include Python3.6 in their packages, but that is not the case. Can you re-explain what you're running into? Thanks |
@iscai-msft Hello, first of all i want to apologize to post a rather unfriendly and not detailed issue.
after
So, if there is no problem, I would like to update the autorest on the metrics advisor, Is it okay? Thanks. Hong |
No worries @wonder6845 , thanks for the clarification! The setup.py that metrics advisor has actually isn't generated by AutoRest. There are some specific dependencies that we've added, so the SDK author currently has full control of the setup.py, and AutoRest generation won't change anything about the setup.py. That being said, I'm not opposed to an AutoRest regeneration PR, it has been a while since we've regenerated, and the generated code will change a decent amount. The setup.py will remain the same though with generation, so if you do regenerate please take care to update the setup.py mindep. This isn't high priority though, thanks for asking! |
Is this still valid? |
Hi @wonder6845. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue. |
Hi @wonder6845, we're sending this friendly reminder because we haven't heard back from you in 7 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you! |
I find a issue during updating about drop python 3.6 at metricsadvisor (ref ##25814)
I think it's better updating min dependency package autorest.
-autorest-
#25731
please inform me if anything problem found
Thank you.
The text was updated successfully, but these errors were encountered: