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

CTX-4070: Streaming download #19

Merged
merged 8 commits into from
Jul 17, 2023
Merged

Conversation

VukManojlovic
Copy link
Collaborator

No description provided.

@VukManojlovic VukManojlovic temporarily deployed to develop June 27, 2023 15:38 — with GitHub Actions Inactive
@VukManojlovic VukManojlovic marked this pull request as draft June 28, 2023 08:37
@igorperic17
Copy link
Collaborator

@VukManojlovic @dule1322 what's the status of this one?

@dule1322
Copy link
Collaborator

@igorperic17 This one remains in draft until backend starts sending the "Content-Length" header on "session/export" endpoint.

@igorperic17
Copy link
Collaborator

What's the ETA on the backend work?

@dule1322
Copy link
Collaborator

dule1322 commented Jul 3, 2023

@igorperic17 The task was in CR, it is merged now so @VukManojlovic can make necessary modifications to this PR:

  • Update it to use streaming download only for session/export endpoint
  • Compare value of "Content-Size" header field with the file locally to re-download the file it it was not fully downloaded (corrupted)

@dule1322
Copy link
Collaborator

dule1322 commented Jul 3, 2023

Update: It is merged but the backend build is not passing, so it is still not in dev environment.

Vuk Manojlovic added 2 commits July 11, 2023 17:25
…implemented checking for downloaded file corruption (if file size does not match expected size)
* develop:
  CTX-4079: Redesigned terminate condition for experiment worker process - Added log file output for experiment worker process for debugging purposes
  CTX-4251: Update "model-queue/custom" to "run" endpoint
  Version bump
  CTX-4219: Make root not private to make it available outside the class - Name starts with an underscore to mark it as internally used don't touch
  CTX-4083: Moved heartbeat to the metrics worker process - Renamed metrics worker process to experiment worker
  Version bump
  CTX-4102: Fix terminating metrics process on experiment process kill
  CTX-4166: Implemented Sequence sample import
  CTX-3890: Changed division to //
  CTX-3890: Converted wider code to be compatible with int based bounding boxes
  CTX-3890: Changed width and height variables from float to int
  Update README.md
@VukManojlovic VukManojlovic temporarily deployed to develop July 11, 2023 15:26 — with GitHub Actions Inactive
@VukManojlovic VukManojlovic marked this pull request as ready for review July 11, 2023 15:28
@VukManojlovic VukManojlovic temporarily deployed to develop July 11, 2023 15:28 — with GitHub Actions Inactive
@VukManojlovic VukManojlovic temporarily deployed to develop July 14, 2023 11:11 — with GitHub Actions Inactive
@VukManojlovic VukManojlovic temporarily deployed to develop July 14, 2023 11:43 — with GitHub Actions Inactive
@igorperic17 igorperic17 marked this pull request as draft July 14, 2023 12:53
@VukManojlovic VukManojlovic temporarily deployed to develop July 14, 2023 13:30 — with GitHub Actions Inactive
@igorperic17 igorperic17 marked this pull request as ready for review July 17, 2023 09:01
@igorperic17 igorperic17 merged commit 43c9eff into coretex-ai:develop Jul 17, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants