clamp counter to actual buffer size #1703
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dkales
previously approved these changes
Oct 2, 2025
eaypek-tfh
approved these changes
Oct 2, 2025
naure
added a commit
that referenced
this pull request
Oct 2, 2025
* do not run against dev (#1695) * Bump actions/checkout from 4.2.2 to 5.0.0 (#1651) Bumps [actions/checkout](https://github.com/actions/checkout) from 4.2.2 to 5.0.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4.2.2...08c6903) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 5.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Krzysztof Szarek <kszarek@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> * Bump tj-actions/changed-files from 46.0.5 to 47.0.0 (#1656) Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 46.0.5 to 47.0.0. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](tj-actions/changed-files@ed68ef8...24d32ff) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-version: 47.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump docker/setup-buildx-action from 3.10.0 to 3.11.1 (#1652) Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 3.10.0 to 3.11.1. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v3.10.0...e468171) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-version: 3.11.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump docker/build-push-action from 6.15.0 to 6.18.0 (#1650) Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.15.0 to 6.18.0. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](docker/build-push-action@v6.15.0...2634353) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-version: 6.18.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> * Ability to disable anon stats per-batch, from SQS messages (#1696) * enable the ability to disable anon stats per-batch, from SQS messages * basic tests for e2e anon stats disabled * test fix * probabilistic disablement of anon stats in tests * remove unnecessary if statement * enable reauth in e2e test (#1694) e2e: enable reauth * Oblivious swap network (#1682) Add oblivious swap network * clamp counter to actual buffer size (#1703) * clamp counter to actual buffer size * warning log if clamping * improve logging for anon stats disabling --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carlo Mazzaferro <carlo.mazzaferro@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Krzysztof Szarek <kszarek@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> Co-authored-by: Wojciech Sromek <157375010+wojciechsromek@users.noreply.github.com> Co-authored-by: iliailia <iliailiashenko@gmail.com>
mcalancea
added a commit
that referenced
this pull request
Oct 3, 2025
* draft naive-knn * Release memory correctly add checkpoints and safeguards * clippy * fixes * change floats to fractions * dev: deploy main * dev: Increase timeout to 30min to support batch_size=32 * [POP-2929] add graceful shutdown to the networking stack (#1685) * squash * dont try to reconnect if sessions are dropped * add more shutdown handlers * add cancellation token to the connection builder * reduce logs * address code review * omit reconnect failure due to cancellation * return error if message fails to deserialize * u32 instead of usize and stream.take * Merge main -> dev (#1705) * do not run against dev (#1695) * Bump actions/checkout from 4.2.2 to 5.0.0 (#1651) Bumps [actions/checkout](https://github.com/actions/checkout) from 4.2.2 to 5.0.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4.2.2...08c6903) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 5.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Krzysztof Szarek <kszarek@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> * Bump tj-actions/changed-files from 46.0.5 to 47.0.0 (#1656) Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 46.0.5 to 47.0.0. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](tj-actions/changed-files@ed68ef8...24d32ff) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-version: 47.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump docker/setup-buildx-action from 3.10.0 to 3.11.1 (#1652) Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 3.10.0 to 3.11.1. - [Release notes](https://github.com/docker/setup-buildx-action/releases) - [Commits](docker/setup-buildx-action@v3.10.0...e468171) --- updated-dependencies: - dependency-name: docker/setup-buildx-action dependency-version: 3.11.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump docker/build-push-action from 6.15.0 to 6.18.0 (#1650) Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.15.0 to 6.18.0. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](docker/build-push-action@v6.15.0...2634353) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-version: 6.18.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> * Ability to disable anon stats per-batch, from SQS messages (#1696) * enable the ability to disable anon stats per-batch, from SQS messages * basic tests for e2e anon stats disabled * test fix * probabilistic disablement of anon stats in tests * remove unnecessary if statement * enable reauth in e2e test (#1694) e2e: enable reauth * Oblivious swap network (#1682) Add oblivious swap network * clamp counter to actual buffer size (#1703) * clamp counter to actual buffer size * warning log if clamping * improve logging for anon stats disabling --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carlo Mazzaferro <carlo.mazzaferro@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Krzysztof Szarek <kszarek@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> Co-authored-by: Wojciech Sromek <157375010+wojciechsromek@users.noreply.github.com> Co-authored-by: iliailia <iliailiashenko@gmail.com> * clippy --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Aurel <naure@users.noreply.github.com> Co-authored-by: sdwoodbury <stuart@inversed.tech> Co-authored-by: Carlo Mazzaferro <carlo.mazzaferro@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Krzysztof Szarek <kszarek@users.noreply.github.com> Co-authored-by: Ertugrul Aypek <ertugrul.aypek@toolsforhumanity.com> Co-authored-by: Wojciech Sromek <157375010+wojciechsromek@users.noreply.github.com> Co-authored-by: iliailia <iliailiashenko@gmail.com> Co-authored-by: Bryan Gillespie <bgillesp@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In response to the observed crash of iris-mpc