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

Custom-vision training iteration JavaScript export #10331

Closed
6 tasks
YoussefSbeiti opened this issue Jul 29, 2020 · 10 comments
Closed
6 tasks

Custom-vision training iteration JavaScript export #10331

YoussefSbeiti opened this issue Jul 29, 2020 · 10 comments
Assignees
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Cognitive - Custom Vision customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention This issue is responsible by Azure service team.

Comments

@YoussefSbeiti
Copy link

  • Package Name: @azure/cognitiveservices-customvision-training
  • Package Version: 5.1.1
  • Operating system: Windows 10
  • nodejs
    • version:
  • browser
    • name/version:
  • typescript
    • version:
  • Is the bug related to documentation in

Describe the bug
When trying to send a second exportIteration() request through the SDK I get the following error:
restError.ts:19 Uncaught Error: {iterationId} is already queued for export

To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.
image

Additional context
Add any other context about the problem here.

@ghost ghost added needs-triage This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Jul 29, 2020
@ramya-rao-a ramya-rao-a added Client This issue points to a problem in the data-plane of the library. Cognitive - Custom Vision labels Jul 29, 2020
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Jul 29, 2020
@ramya-rao-a ramya-rao-a added the Service Attention This issue is responsible by Azure service team. label Jul 29, 2020
@ghost
Copy link

ghost commented Jul 29, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @areddish, @tburns10.

@ghost ghost added the needs-team-attention This issue needs attention from Azure service team or SDK team label Jul 29, 2020
@ramya-rao-a
Copy link
Contributor

Thanks for reporting @YoussefSbeiti

@areddish, @tburns10, Can you clarify if this is expected behavior?

cc @sarangan12 in case the service team needs any follow up from the client side

@areddish
Copy link
Contributor

@ramya-rao-a Yes, this is expected behavior.

@YoussefSbeiti exporting an iteration is an asynchronous process, you are getting this error because exportIteration was already called on the iteration and it's still queue'd or currently in the process of being exported. We don't allow multiple exports of the same iteration to happen in parallel, you have to wait for the currently queue'd export to finish.

You can get the status of the exports for that specific iteration by calling getExports with the project id and iteration id you are interested in. One of the fields returned will be status and that contains information to let you know if the export is queued, completed, failed, etc.

@YoussefSbeiti
Copy link
Author

Thank you for the reply @areddish @ramya-rao-a .
I still get the same Uncaught Error even though the status of the previous request is "Done"
image

@andreasbotsikas
Copy link

Even with python SDK, if you try to export an iteration which has already been exported ("Done" in the list) we get the same error:

{iterationId} is already queued for export

A different error should be emitted to be able to distinguish that the model is already available in the exported list.
FYI @OlivelliAri

@areddish
Copy link
Contributor

Thanks - yes, this is an error in the error message that get's generated. I've logged an issue in our internal task system to provide a better error message and will address in a future roll out.

@ramya-rao-a ramya-rao-a added this to the Backlog milestone Oct 2, 2020
@ramya-rao-a ramya-rao-a added bug This issue requires a change to an existing behavior in the product in order to be resolved. and removed needs-team-attention This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Oct 2, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Dec 30, 2020
Api Management - make /tenant endpoints ARM compliant in 2020-06-01-preview version (Azure#11549)

* Adds base for updating Microsoft.ApiManagement from version stable/2019-12-01 to version 2020-06-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add support in API Management for Availability Zones (Azure#10284)

* apim in azs

* fix prettier check

* PATCH should return 200 OK (Azure#10328)

* add support for PATCH returning 200 OK

* CI fixes

prettier fix

CI fixes part 2

* Password no longer a mandatory property when uploading Certificates

* add missing x-ms-odata extension for filter support

* +gatewayhostnameconfiguration protocol changes (Azure#10292)

* [2020-06-01-preview] Update Oauth Server secrets Contract (Azure#10602)

* Oauth server secrets contract

* fix azureMonitor enum

* API Management Service Deleted Services Resource (Azure#10607)

* API Management Service Deleted Services Resource

* Path fix

* Lint + custom-words fixes

* Location URI parameter for deletedservices Resource

* GET for deletedservices by service name

* Remove resourceGroupName from resource path

* fixes

* schema for purge operation

* perttier applied

* 204 response code added

Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>

* OperationNameFormat property added to Diagnostic contract (Azure#10641)

* OperationNameFormat property added to Diagnostic contract

* add azuremonitor to update contract

Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>

* [Microsoft.ApiManagement][2020-06-01-preview] Change Network Status response contract (Azure#10331)

* Change Network Status response contract

* Update examples for network status contract

* ApiManagement - tenant/settings endpoints

* ApiManagement - tenant/settings endpoints fix

* ApiManagement - tenant/settings endpoints fix prettier

* ApiManagement - tenant/settings endpoints fix 3

* ApiManagement - tenant/settings endpoints fix 4

* ApiManagement - tenant/settings endpoints fix 5

Co-authored-by: Samir Solanki <samirsolanki@outlook.com>
Co-authored-by: maksimkim <maksim.kim@gmail.com>
Co-authored-by: promoisha <feoktistovgg@gmail.com>
Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>
Co-authored-by: RupengLiu <rliu1211@terpmail.umd.edu>
Co-authored-by: vfedonkin <vifedo@microsoft.com>
@ramya-rao-a
Copy link
Contributor

I've logged an issue in our internal task system to provide a better error message and will address in a future roll out.

Hey @areddish,

This package had an update in Nov 2020 with the changes from #12427. Did we get the error message updated as part of this release?

@agribot2
Copy link

agribot2 commented Apr 6, 2022

Please provide a fix for this defect. I do believe it is only a problem with the error message wording. I believe that there is also a functional ( code ) defect that is causing the system to invoke an error message.

@sarangan12
Copy link
Member

@areddish Do we have an update on this issue?

Copy link

Hi @YoussefSbeiti, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 20, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 20, 2024
@xirzec xirzec removed this from the Backlog milestone May 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. Cognitive - Custom Vision customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

7 participants