Skip to content

Note about large file uploads and cancel button#83

Merged
VasileiaVagena merged 2 commits intomainfrom
upload-files-tip
Mar 27, 2026
Merged

Note about large file uploads and cancel button#83
VasileiaVagena merged 2 commits intomainfrom
upload-files-tip

Conversation

@VasileiaVagena
Copy link
Copy Markdown
Collaborator

  • add instruction for cancel-upload button
  • add note related to large file uploads (prefer wired internet connection) (#40)
  • add new updated GIF

- add note related to large files uploads (prefer wired internet connection)
- add new updated GIF
@github-actions
Copy link
Copy Markdown

🚀 PR Preview deployed!

📖 Preview your changes: https://biosustain.github.io/data-catalog-resources/pr-83-608399958b8766ff/

🔒 Security Note: This URL contains a random component for security

This preview will be updated automatically when you push new commits.

github-actions bot pushed a commit that referenced this pull request Mar 25, 2026
@VasileiaVagena VasileiaVagena requested a review from dinhe878 March 25, 2026 15:16
@VasileiaVagena
Copy link
Copy Markdown
Collaborator Author

@dinhe878 Could you please review this PR?
Based on upload tests with different file sizes #40
I believe a practical workaround, at least for now, is for the users to use a wired internet connection instead of WiFi when uploading large files (5-10GB up to 50GB).

Comment thread docs/Dataset/Files.md



```{note}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@pcolaianni could you verify this statement?

Copy link
Copy Markdown
Collaborator

@dinhe878 dinhe878 left a comment

Choose a reason for hiding this comment

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

I will also ask @pcolaianni to review this pr

@dinhe878 dinhe878 requested a review from pcolaianni March 26, 2026 07:30
Copy link
Copy Markdown
Member

@pcolaianni pcolaianni left a comment

Choose a reason for hiding this comment

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

I smoothed out the sentence with ChatGPT. What do you think?

Comment thread docs/Dataset/Files.md Outdated


```{note}
The system queues files upon upload, meaning they are uploaded sequentially rather than in parallel. Because of this, the total upload time depends almost entirely on the network upload speed during the transfer. For large uploads (5–10 GB up to 50 GB), avoid Wi‑Fi and use a **wired internet connection** for much faster and more stable upload speeds.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
The system queues files upon upload, meaning they are uploaded sequentially rather than in parallel. Because of this, the total upload time depends almost entirely on the network upload speed during the transfer. For large uploads (5–10 GB up to 50 GB), avoid Wi‑Fi and use a **wired internet connection** for much faster and more stable upload speeds.
Files are queued on upload, meaning they transfer sequentially rather than in parallel. Total upload time depends entirely on your network's upload speed. For large transfers (5+ GB), use a wired connection instead of Wi-Fi to achieve faster and more stable uploads.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I intentionally removed some parts from the sentences. Such as "almost", or "because of this", etc.

That is because either sequential or parallel, the network speed affects both the same way.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks @pcolaianni for reviewing this and for the suggestion! I have committed it.

@github-actions
Copy link
Copy Markdown

🚀 PR Preview deployed!

📖 Preview your changes: https://biosustain.github.io/data-catalog-resources/pr-83-cb075b9a85f139a8/

🔒 Security Note: This URL contains a random component for security

This preview will be updated automatically when you push new commits.

github-actions bot pushed a commit that referenced this pull request Mar 27, 2026
@dinhe878
Copy link
Copy Markdown
Collaborator

@VasileiaVagena looks fine. Feel free to merge.

@VasileiaVagena VasileiaVagena merged commit f8de7ec into main Mar 27, 2026
3 checks passed
@VasileiaVagena VasileiaVagena deleted the upload-files-tip branch March 27, 2026 10:46
@github-actions
Copy link
Copy Markdown

🧹 PR Preview Cleanup

Preview directories for this PR have been automatically removed from the deployment.

This helps keep the repository clean by removing old preview deployments.

github-actions bot pushed a commit that referenced this pull request Mar 27, 2026
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