Repository navigation
4/8 Precision pathways and crissCrossValidate fixes (cumulative costs, honest diagonal) - #129
Merged
Merged
Conversation
…dict with stored models - .permutations keeps a matrix (drop = FALSE), so clinical plus one assay works in the default mode. - fixedAssays may be NULL or several assays; list input no longer calls colData(). - Models are named and looked up by assay; prediction uses each result's own predictParams. - precisionPathwaysPredict predicts every new sample with every stored fold model instead of re-splitting the new cohort with runTests; classes of new samples are optional. - A sample is charged for every tier it passed through (cumulative cost). - summary() matches weights by name and checks they sum to 1. - The confidence cut-off is a minimum (>=, with a floating point tolerance). - Each pathway carries its assay sequence; no strsplit on '-'. - bubblePlot and strataPlot no longer call theme_set(); bubblePlot keeps pathways below 0.5; strataPlot draws the true-class strip as its own layer. - Fold models drop the forImportance forests, which prediction does not use. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
…nly when used - TOP is required only when runTOP = TRUE. - Data sets are restricted to shared features (with a message) before training and testing. - The modelTest diagonal is the crossValidate run on that data set, not a leaky re-test. - Outcome type is checked for every data set; performanceType is resolved before tuning params. - Random features use max(nFeatures) when several values are tuned. - crissCrossPlot no longer attaches the result, labels the diagonal, and returns the plot. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
…nctions, show fixes
- PredictParams on a one-step classifier or unknown keyword gives a clear error.
- SelectParams('none') returns NULL; SelectParams accepts plain ranking functions.
- TrainParams(balancing = ) is an error pointing to ModellingParams.
- show methods no longer error on missing characteristics and end lines with newlines.
- Remove the unused misspelled class union; fix the 'higer' typo for AUC.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
…orrect source URL Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
…ix PredictParams show test Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
…n from a saved model Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4
This was referenced Oct 9, 2026
Merged
DarioS
approved these changes
Oct 28, 2026
DarioS
left a comment
Member
There was a problem hiding this comment.
Great speed improvements. Confidence is indeed repeated consistency - not accuracy.
DarioS
marked this pull request as ready for review
October 28, 2026 02:00
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.
Summary
Fixes to precision pathways,
crissCrossValidateand the parameter classes.Why and what
Precision pathways:
>=), as documented.fixedAssays = NULLand list input work;summary()matches weights by name;-work;bubblePlotno longer silently drops pathways below 0.5 accuracy.crissCrossValidate:
modelTestdiagonal leaked test samples into feature selection. On noise it gave balanced accuracy 0.85, where honest CV gives 0.51. It is now each data set's own cross-validation, labelled as such.runTOP = TRUE.Classes:
SelectParams("none")and plain functions work;TrainParams(balancing = )is an error pointing toModellingParams(it was silently ignored);Do results change?
Yes:
modelTestdiagonal changes.Evidence
289 tests pass (
test-pathways.R,test-crisscross.R,test-classes.R).Questions
minAssaySamplesat prediction makes one patient's routing depend on the rest of the batch.🤖 Generated with Claude Code
https://claude.ai/code/session_01TfsjwkbDbHkX4kJZNe4ER4