Skip to content

fix: raise MAX_FILE_SIZE_GB to the documented 100 GB limit - #630

Open
Kayvan-Zahiri wants to merge 1 commit into
togethercomputer:mainfrom
Kayvan-Zahiri:fix/max-file-size-100gb
Open

Kayvan-Zahiri wants to merge 1 commit into
togethercomputer:mainfrom
Kayvan-Zahiri:fix/max-file-size-100gb

Conversation

@Kayvan-Zahiri

Copy link
Copy Markdown

The fine-tuning docs say we accept files up to 100 GB, but check_file still used MAX_FILE_SIZE_GB = 50.1. A 60 GB file failed the size check even though it's under the documented limit.

I bumped the constant to 100 so it matches the docs. NUM_BYTES_IN_GB is still 2**30, so this is 100 GiB the same way the old 50.1 value was. Added a test that a 60 GB file now passes, and one that a file over the cap still fails.

Fixes #588

Made with Cursor

check_file still rejected files above 50.1 GB even though the fine-tuning docs advertise 100 GB.

Co-authored-by: Cursor <cursoragent@cursor.com>
@broly-code-security-scanner

broly-code-security-scanner Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Broly Security Scan

Warning

Latest baseline snapshot is stale. Broly is running in PR-only fallback mode until the next scheduled baseline refreshes. This does not block the PR.

Note

Re-scan this PR anytime with /broly scan — useful after /broly undismiss, or to refresh findings without a new push.

Broly — SAST (GLM-5.3-Flash) · Secrets · SCA · IaC · GH Actions · Base Images · Supply Chain Threats · Attack Hypotheses · Adversarial Verification

We're continuously improving Broly's accuracy and finding quality — your feedback is valuable. False positives, missed findings, bugs, and feature requests all welcome.

Ask in #security-engineering   Powered by Together AI

This branch has not been deployed

No deployments
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.

Fine-tuning file size guard rejects 60 GB despite documented 100 GB limit

1 participant