Repository navigation
Fix compact result imports and shallow Plate reanalysis - #55
Merged
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
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.
Local NL-BIOMERO #52 acceptance exposed two failures: a valid 417-byte CellExpansion result archive was rejected before import, and using an already-shallowed Plate as native Zarr input raised
AttributeErrorbecause transfer expected the old materializer collection field.Validate ZIP contents without the arbitrary 1 KB minimum; existing missing, empty, directory-only and invalid-ZIP checks remain. Rebuild Plate canonical inputs from the schema-2 manifest graph and source bindings, retaining managed labels and compatibility with the older materializer representation. Align all script versions to the coordinated BIOMERO 2.9.0b17 prerelease.
Validation: positive regressions reproduced both failures before the changes; six archive checks and five Plate reanalysis checks pass afterward, including v2/v3, retained labels and rejected missing labels. The focused slice passes 75 tests. Tests remain on the separate harness branch, using the published shallower return type. Full harness and live reruns are the release gates.
Merge the companion capability-gated harness PR into
test-suite, rerun this PR's CI against that harness, then merge this PR and publish scripts beta.17. The final assembled-stack evidence belongs to NL-BIOMERO #52.