Skip to content

Conversation

@Dhruv-Mehndhiratta-Centml
Copy link
Contributor

  • currently we have attached the hardware instance mapping api call to decorator function to fetch common click utils which are common in inference and compute deployment but because its a decorator function its trying to make the api call when we are executing the centml command, its working perfectly fine for the user who is already loggedin but for the user who is logged out and when the same user try to see the centml options its making hardware instance api call and eventually leading it to fail the api call
  • in order to fix the same now we are going the load the choices of hardware instance mapping in a deferred way and if incase user enters any incorrect pricing tier it will result to the user to provide the same listing options so that user can only enter allowed values

@Dhruv-Mehndhiratta-Centml Dhruv-Mehndhiratta-Centml force-pushed the bugfix/fix-common-decorator-function-api-defer-call branch from 7a1c995 to 49f9c80 Compare July 2, 2024 16:46
@Dhruv-Mehndhiratta-Centml Dhruv-Mehndhiratta-Centml added the bug Something isn't working label Jul 2, 2024
@Dhruv-Mehndhiratta-Centml Dhruv-Mehndhiratta-Centml force-pushed the bugfix/fix-common-decorator-function-api-defer-call branch from 410c262 to 6f79c04 Compare July 5, 2024 16:33
@Dhruv-Mehndhiratta-Centml Dhruv-Mehndhiratta-Centml deleted the bugfix/fix-common-decorator-function-api-defer-call branch July 5, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants