Delete the cost sweep's wall-clock figures instead of re-measuring them - #496
Merged
Merged
Conversation
…measured The sweep priced barcode correction and UMI collapse against a one-entry list, so its wall clocks are floors, not slopes. Nothing rests on them, so the column and the ~21 s + 3 s per million reads fit are gone instead of scheduled for a re-run: a number no decision reads is not worth an arc job. The memory table stays. The omitted work does not grow with read depth, so flat-at-the-index survives its own instrument's defect, and the intercept omits only what a 6.8M-entry list costs STARsolo. The caveat shrinks to match -- it no longer defers a judgement to a re-run that will not happen.
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.
Closes #495.
The sweep's wall clocks were taken while barcode correction and UMI collapse had a one-entry list to work against (#493), so they are floors rather than slopes. No decision reads them, so they are deleted rather than scheduled for an arc re-run.
STAR wallcolumn and the~21 s + 3 s per million readsfit.No pipeline behaviour changes; this is the research doc only.