Azure/azure-rest-api-specs

[Cognitive Services - Azure AI Services] API Review

azure-sdk opened this issue · 6 comments

New API Review meeting has been requested.

Service Name: Cognitive Services - Azure AI Services
Review Created By: Dapeng Li
Review Date: 05/30/2024 04:00 PM PT
Release Plan:
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided

Description: Video dubbing API to translate video from source locale to target locale.

Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Dapeng Li

Service Name: Cognitive Services - Azure AI Services
Review Created By: Dapeng Li
Review Date: 05/30/2024 04:00 PM PT
Release Plan:
PR:
Hero Scenarios Link: here
Core Concepts Doc Link: Not Provided

Description: Video dubbing API to translate video from source locale to target locale.
Latest:
https://github.com/Azure/azure-rest-api-specs-pr/pull/18399

Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Notes from API Review meeting 5/30/24

  • For long-running delete, does GET still work while delete is running? What about create with same name?
    • we want to avoid "ghosts". Resource should show as available until delete completes
    • Or make the delete synchronous with clean up handled silently in the background
  • Consider cost of storage
  • Please consider moving the definition to TypeSpec
  • What happens if the user does a PUT for an id that already exists?
    • Same input, return a 201, and not same input returns an error
  • What happens when the source content at the URL changes ?
    • Might need to capture ETag from source URL and fail
  • PUT is naturally idempotent
  • Consider not supporting anything but managed identity auth
  • Create of translation also creates the first iteration -- that's a bit odd
    • But it avoids needing two calls to create first iteration
    • Also subsequent iterations are cheaper
  • What happens if I try to create a new iteration when the last one is still running? Error
  • LastCompletedIteration -> LastSucceededIteration
  • How does the billing work when the initial translation failed?

Please convert to TypeSpec, in the process address whatever issues above that you can, and then schedule another meeting.

Meeting updated by Dapeng Li

Service Name: Cognitive Services - Azure AI Services
Review Created By: Dapeng Li
Review Date: 05/30/2024 04:00 PM PT
Release Plan:
PR:
Hero Scenarios Link: here
Core Concepts Doc Link: Not Provided

Description: Video dubbing API to translate video from source locale to target locale.
Latest:
https://github.com/Azure/azure-rest-api-specs-pr/pull/18399

Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

New API Review meeting has been requested.

Service Name: Cognitive Services - Azure AI Services
Review Created By: Dapeng Li
Review Date: 06/13/2024 04:00 PM PT
Release Plan:
PR:
Hero Scenarios Link: here
Core Concepts Doc Link: Not Provided

Description: Continue to review video translation API:
https://github.com/Azure/azure-rest-api-specs-pr/pull/18399

Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Cancelled by: Dapeng Li

Notes from API Review meeting 5/30/24

  • For long-running delete, does GET still work while delete is running? What about create with same name?

    • we want to avoid "ghosts". Resource should show as available until delete completes
    • Or make the delete synchronous with clean up handled silently in the background
  • Consider cost of storage

  • Please consider moving the definition to TypeSpec

  • What happens if the user does a PUT for an id that already exists?

    • Same input, return a 201, and not same input returns an error
  • What happens when the source content at the URL changes ?

    • Might need to capture ETag from source URL and fail
  • PUT is naturally idempotent

  • Consider not supporting anything but managed identity auth

  • Create of translation also creates the first iteration -- that's a bit odd

    • But it avoids needing two calls to create first iteration
    • Also subsequent iterations are cheaper
  • What happens if I try to create a new iteration when the last one is still running? Error

  • LastCompletedIteration -> LastSucceededIteration

  • How does the billing work when the initial translation failed?

Please convert to TypeSpec, in the process address whatever issues above that you can, and then schedule another meeting.
image