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

Feature/upload images background #76

Merged
merged 2 commits into from
Aug 1, 2022

Conversation

leihuayi
Copy link
Contributor

@leihuayi leihuayi commented Aug 1, 2022

This feature works also as a fix for the issue of swiftclient sometimes failing to upload images to OVH.
Instead of the previous workflow:

  1. user uploads image to client
  2. client (frontend) sends image as bytes to api (backend)
  3. backend uploads bytes image to ovh swift
  4. meanwhile, backend processes bytes image with ML algorithm
  5. when the upload and ML analysis are finished, return result and url to location of image in swift
  6. display in client (frontend) url of image in swift

Now do:

  1. user uploads image to client
  2. client (frontend) sends image as bytes to api (backend)
  3. backend processes bytes image with ML algorithm
  4. when the ML analysis are finished, return result
  5. backend then uploads image to swift as a background process
  6. meanwhile, the client has received the response from backend and displays image using the byte data originally uploaded by user

@github-actions github-actions bot added the feature New feature / functionality label Aug 1, 2022
@leihuayi leihuayi merged commit d4da162 into develop Aug 1, 2022
@leihuayi leihuayi deleted the feature/upload-images-background branch August 1, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature / functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant