Skip to content

fix: Turn off TIFF chunking for now#295

Merged
kylebarron merged 1 commit intomainfrom
kyle/remove-chunk-size-fromurl
Mar 3, 2026
Merged

fix: Turn off TIFF chunking for now#295
kylebarron merged 1 commit intomainfrom
kyle/remove-chunk-size-fromurl

Conversation

@kylebarron
Copy link
Copy Markdown
Member

Ref #294

This was causing all our requests to be made in 32kb increments, totally ballooning the number of requests we were making. @gadomski hit this last week, and hopefully after this his demo should be much faster.

@kylebarron kylebarron marked this pull request as ready for review March 3, 2026 00:46
@kylebarron kylebarron enabled auto-merge (squash) March 3, 2026 00:46
@github-actions github-actions bot added the fix label Mar 3, 2026
@kylebarron kylebarron merged commit c4996b2 into main Mar 3, 2026
3 checks passed
@kylebarron kylebarron deleted the kyle/remove-chunk-size-fromurl branch March 3, 2026 00:47
@ds-release-bot ds-release-bot bot mentioned this pull request Mar 3, 2026
@gadomski
Copy link
Copy Markdown
Contributor

gadomski commented Mar 3, 2026

Incredibly better:

Screen.Recording.2026-03-03.at.4.10.53.AM.mov

@kylebarron
Copy link
Copy Markdown
Member Author

@gadomski that looks awesome. Do you know if maybe some of the requests were already cached? Or you just have a crazy fast internet?

@gadomski
Copy link
Copy Markdown
Contributor

gadomski commented Mar 3, 2026

Or you just have a crazy fast internet?

Municipal gigabit fiber so yeah :-)

@kylebarron
Copy link
Copy Markdown
Member Author

Indeed that is faster than the office...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants