Repository navigation
Update NEXT_TASKS for merged PR #916 - #952
Merged
Merged
Conversation
Collaborator
Author
|
Review finding: #953. Triaged in scope because the new delivery summary overstates retained model-byte provenance and independent measurement coverage. The correction will stay within the assessment paragraph; the seven scientific follow-up dispositions and underlying evidence remain unchanged. Fresh-context review is also in progress. |
Collaborator
Author
|
Final review/validation handoff at be8ef25:
PR left OPEN and unmerged for the user's explicit decision. The secondary is idle; the branch is clean and fully pushed. |
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.
NEXT_TASKS.md still described the ProSMART correction, X-ray assessment and tools guide as local work awaiting merge. This records their delivery through PR #916 and squash 0320d1a, links the closed issues, and updates the dated pre-PR queue snapshot. The seven open scientific issue dispositions remain byte-for-byte unchanged; the cryo-EM extension remains proposed work.
Fresh-context adversarial review found the assessment summary overstated per-model byte provenance. Issue #953 records the finding and its in-scope triage. The five-line fix distinguishes verified retained 1SAR inputs from fixed control JSON summaries and preserves historical selection, raw-log and independent-coverage limits. A separate fresh-context review of the fix found no actionable findings.
Validation on final head be8ef25:
uv run --locked --extra benchmark -- bash scripts/validate.shwith Python 3.12.11: exit 0 in 136.06 seconds; clean, unchanged head afterward.Closes #953.
The existing scientific follow-ups #760, #765, #790, #799, #872, #873 and #876 remain open. Separate local Reduce/Probe version-and-hash verification is recorded in orchestration and is outside this documentation diff. No scientific campaign was run. Merge awaits the user's explicit go-ahead for this PR.