Skip to content

Conversation

@ztorage
Copy link

@ztorage ztorage commented Mar 6, 2025

No description provided.

s3/src/bucket.rs Outdated
use http::HeaderMap;

pub const CHUNK_SIZE: usize = 268_435_456; // 256 Mebibytes (8 wasn't enough), min is 5 (5_242_880);
pub const CHUNK_SIZE: usize = 16_777_216; // 16 Mebibytes (8 wasn't enough), min is 5 (5_242_880);

Choose a reason for hiding this comment

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

This block size is needed to support uploading 70B models since S3 protocol only allows a max of 10k blocks.

use http::HeaderMap;

pub const CHUNK_SIZE: usize = 268_435_456; // 256 Mebibytes (8 wasn't enough), min is 5 (5_242_880);
pub const CHUNK_SIZE: usize = 33_554_432; // 32 Mebibytes (8 wasn't enough), min is 5 (5_242_880);
Copy link

Choose a reason for hiding this comment

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

what model was this tested with?

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.

6 participants