Repository navigation
Allow explicitly larger compact input exports without changing default bounds - #607
Open
CaelmBleidd wants to merge 1 commit into
Open
CaelmBleidd wants to merge 1 commit into
CaelmBleidd wants to merge 1 commit into
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
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.
The 30-second calibration generated 13–14k inputs but retained only 3956 under the 2 MiB export limit, making complete class-coverage comparison ineligible. Allow an explicit input-export byte limit up to 64 MiB while keeping its default at 2 MiB and the invocation bound at 100k. Increase the private stdout allowance only by the extra bytes explicitly requested for that export; ordinary requests keep the existing 4 MiB stdout, 4 MiB input and 64 KiB stderr limits. The shared transport API and timeout behavior are unchanged.
The experimental driver accepts the same explicit bound. Sampling quotas remain independent of capture limits, and omissions still disqualify complete coverage. No spooling, sessions or persistent capture service is added. A small style correction from the current coordinator branch is included in the existing guided test.
Stacked directly on #597
b09a010f8, independently of the numeric coordinator.Validation: a real native/backend regression returns over 4 MiB of compact inputs alongside coverage and point observations, preserves 600 inputs including lone UTF-16 surrogates with an explicit 8 MiB limit, and demonstrates retained default overflow on the same values. Eight input-export tests, the generation-deadline regression, process-client boundary tests, 220 Node tests, nine Python tests and main/test Detekt passed. Repeat actual-budget calibration with the new frozen limit before a broad comparison.