Skip to content

Conversation

@ddey2
Copy link
Member

@ddey2 ddey2 commented Jun 21, 2024

No description provided.

@ddey2 ddey2 requested a review from longshuicy as a code owner June 21, 2024 00:27
@ddey2 ddey2 linked an issue Jun 21, 2024 that may be closed by this pull request
@ddey2 ddey2 requested review from Vismayak, lmarini and tcnichol June 21, 2024 00:28
@tcnichol
Copy link
Contributor

@ddey2

I am testing this for datasets and files.

With datasets, when I click the download button the download increments, but the dataset did not download. This happens both on the regular dataset view and the public dataset view.

Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

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

I don't think download works. I see it in the network with the binaries; but since you are not catch the binary and save it to blob then generate a tag, the binary is not accessible to end user.

The public download doesn't work for the same reason.

image

We used to take similar approach as I described before, but at one point we switch to just embed the url and let the browser handle download due to large files. Not sure if we would like to switch back. Maybe something we can discuss?

@ddey2 ddey2 requested a review from longshuicy June 27, 2024 19:27
@ddey2
Copy link
Member Author

ddey2 commented Jun 27, 2024

this is fixed. @tcnichol @longshuicy feel free to take a look

@tcnichol
Copy link
Contributor

Tested.

From the dataset page, every click of download increments. Same for files.

If I download datasets from 'explore' download also increments. Same with downloading files from the filelist.

Marking this one approved.

Copy link
Contributor

@tcnichol tcnichol left a comment

Choose a reason for hiding this comment

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

Download increments on datasets and files from any location. Marking this approved.

Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

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

Works well. Thanks!

@longshuicy longshuicy merged commit e5ed21e into main Jun 27, 2024
@longshuicy longshuicy deleted the 1026-download-incremental-is-not-immediately-reflected branch June 27, 2024 20:20
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.

Download incremental is not immediately reflected

4 participants