Skip to content

enhance downloading APIs#2242

Merged
wyli merged 2 commits intoProject-MONAI:devfrom
wyli:enhance-downloading-apis
May 25, 2021
Merged

enhance downloading APIs#2242
wyli merged 2 commits intoProject-MONAI:devfrom
wyli:enhance-downloading-apis

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented May 24, 2021

Signed-off-by: Wenqi Li [email protected]

  • enhance the download APIs with better default path values
  • remove duplicated check_hash in download_and_extract

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • New tests added to cover the changes.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

@wyli wyli requested a review from Nic-Ma May 24, 2021 21:23
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented May 24, 2021

/integration-test
/black

Copy link
Copy Markdown
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me except for minor comments.

Thanks.

@wyli wyli enabled auto-merge (squash) May 25, 2021 07:43
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli force-pushed the enhance-downloading-apis branch from d877e73 to ff42ae3 Compare May 25, 2021 07:54
@wyli wyli merged commit c3a5bdd into Project-MONAI:dev May 25, 2021
wyli added a commit that referenced this pull request May 26, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 26, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 26, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 26, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 26, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 27, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 27, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request May 27, 2021
* enhance downloading APIs

Signed-off-by: Wenqi Li <[email protected]>

* update docstrings

Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli deleted the enhance-downloading-apis branch June 7, 2021 23:39
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.

2 participants