-
Notifications
You must be signed in to change notification settings - Fork 620
LZ4 0.12 (fix decoding of unsually large chunks) #11533
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Web viewer failed to build. | Result | Commit | Link | Manifest | View image diff on kitdiff. Note: This comment is updated whenever you push a commit. |
4e98906 to
da45a54
Compare
|
@rerun-bot full-check |
|
Started a full build: https://github.com/rerun-io/rerun/actions/runs/18501698194 |
|
Patch has been merged upstream, waiting for a release to be published. |
|
converting to draft so this doesn't show up on review queue |
This reverts commit f593c53.
502d4ff to
ef46e6c
Compare
ef46e6c to
bfec5f3
Compare
This is a follow up to #11525. It fixes the remaining half of #11516. We don't want to merge it as is yet. Let's see if we can get the fix merged upstream first, so we don't have to depend on yet another fork: * PSeitz/lz4_flex#192 --- * Fixes #11516 --- Checks: * [x] `rerun` with `--nb_scans 3000` now works * [x] `rerun rrd stats` with `--nb_scans 3000` now works
This is a follow up to #11525. It fixes the remaining half of #11516.
We don't want to merge it as is yet. Let's see if we can get the fix merged upstream first, so we don't have to depend on yet another fork:
capacity overflow#11516Checks:
rerunwith--nb_scans 3000now worksrerun rrd statswith--nb_scans 3000now works