Skip to content

feat(Downloads): Improve efficiency and functionality - #79

Merged
jhodges10 merged 1 commit into
developfrom
jh/new-download-improvements
Jul 31, 2021
Merged

feat(Downloads): Improve efficiency and functionality#79
jhodges10 merged 1 commit into
developfrom
jh/new-download-improvements

Conversation

@jhodges10

@jhodges10jhodges10 commented Jun 8, 2021

Copy link
Copy Markdown
Contributor

DEVREL-643

Description:

  • Adds optional checksum verification for downloads
  • Adds optional replace boolean for replacing assets at download
  • Improves RAM usage for single-threaded downloads
  • Add a bypass to the multi-part download path for assets under 25 MB since it may ultimately be slower

Depends on:

Includes changes from:

Feedback requested:

  • I haven't really tested this much, we should confirm it works as expected.

@jhodges10
jhodges10 requested a review from lezoudaliJune 8, 2021 02:07
Comment threadframeioclient/lib/download.py
Comment threadframeioclient/lib/download.py
@jhodges10jhodges10 added the enhancement New feature or request label Jun 10, 2021
@jhodges10
jhodges10force-pushed the jh/new-download-improvements branch 2 times, most recently from 9be0bed to c367473CompareJuly 31, 2021 07:33
Add helper functions to simplify a couple of tasks
Fix FrameioHelpers class
Fix FrameioHelpers import
Fix __init__ imports
Finish fixing helpers...
Don't do multi-part downloads on files smaller than 25 MB
Add download improvements from strombergdev
Fix extra prefix in client.assets.download()
Tweak download function
Fix missing call to _get_path() to set self.destination
Add test to Makefile
Fix download function's keyword via ** keyword args
Don't do multi-part downloads on files smaller than 25 MB
@jhodges10
jhodges10force-pushed the jh/new-download-improvements branch from c367473 to c302ec0CompareJuly 31, 2021 07:39
@jhodges10
jhodges10 merged commit de74b81 into developJul 31, 2021
@jhodges10
jhodges10 deleted the jh/new-download-improvements branch July 31, 2021 07:43
@jhodges10jhodges10 mentioned this pull request Aug 3, 2021
@jhodges10jhodges10 changed the title Download functionality improvementsfeat(Downloads): Improve efficiency and functionalityAug 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jhodges10@lezoudali