From 4695c3422bace702e403193f7d0a487d0239f820 Mon Sep 17 00:00:00 2001 From: axiomlogicnexus Date: Tue, 30 Jun 2026 05:11:18 +0000 Subject: [PATCH] Factor HyperTwist recommendation outcome accumulators --- .../HyperTwistTrainingRepositoryLibrary.cpp | 210 ++++++++++-------- ...RTWIST_REFACTORING_TOOLCHAIN_2026-06-22.md | 23 ++ .../HyperTwist/DEVELOPMENT.md | 23 ++ 3 files changed, 162 insertions(+), 94 deletions(-) diff --git a/UnrealHyperTwist/Source/UnrealHyperTwist/Private/HyperTwistTraining/HyperTwistTrainingRepositoryLibrary.cpp b/UnrealHyperTwist/Source/UnrealHyperTwist/Private/HyperTwistTraining/HyperTwistTrainingRepositoryLibrary.cpp index d321278..7a65aad 100644 --- a/UnrealHyperTwist/Source/UnrealHyperTwist/Private/HyperTwistTraining/HyperTwistTrainingRepositoryLibrary.cpp +++ b/UnrealHyperTwist/Source/UnrealHyperTwist/Private/HyperTwistTraining/HyperTwistTrainingRepositoryLibrary.cpp @@ -24217,6 +24217,22 @@ FHyperTwistTrainingCoachRecommendationHistorySummary UHyperTwistTrainingReposito 0.0f; // Preserve the sparse accepted-aligned exact-label mapping while routing // the repeated suffix parsing through one bounded dispatch seam. + auto AccumulateOutcomeHistory = []( + const bool bCondition, + int32& HistoryCount, + float& OutcomeAccumulator, + const float OutcomeScore + ) + { + if (!bCondition) + { + return; + } + + HistoryCount += 1; + OutcomeAccumulator += OutcomeScore; + }; + auto AccumulateAcceptedAlignedTightReliableExactOutcome = [&]( const int32 ReliableSuffixCount, const float OutcomeScore @@ -24659,102 +24675,108 @@ FHyperTwistTrainingCoachRecommendationHistorySummary UHyperTwistTrainingReposito ReliableAlignedTrustedBroadRecommendedLaneOutcomeAccumulator += OutcomeSummary.OutcomeScore; } } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAlignedTightFollowUpPacketStructureCoachMemorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AlignedTightFollowUpPacketStructureOutcomeHistoryCount += 1; - AlignedTightFollowUpPacketStructureOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAlignedBroadRecommendedPacketStructureCoachMemorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AlignedBroadRecommendedPacketStructureOutcomeHistoryCount += 1; - AlignedBroadRecommendedPacketStructureOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedTightFollowUpPacketOutcomeSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketOutcomeHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedBroadRecommendedPacketOutcomeSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketOutcomeHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedTightFollowUpPacketOutcomeTrustSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketTrustOutcomeHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketTrustOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedBroadRecommendedPacketOutcomeTrustSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketTrustOutcomeHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketTrustOutcomeAccumulator += OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketOutcomeTrustSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketTrustReliableOutcomeHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketTrustReliableOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketOutcomeAcceptedHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketTrustReliableLaneOutcomeHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketTrustReliableLaneOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketOutcomeTrustSource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableOutcomeHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketTrustReliableOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketOutcomeAcceptedHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableLaneOutcomeHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketTrustReliableLaneOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactAcceptedHistorySource( + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAlignedTightFollowUpPacketStructureCoachMemorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AlignedTightFollowUpPacketStructureOutcomeHistoryCount, + AlignedTightFollowUpPacketStructureOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAlignedBroadRecommendedPacketStructureCoachMemorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AlignedBroadRecommendedPacketStructureOutcomeHistoryCount, + AlignedBroadRecommendedPacketStructureOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedTightFollowUpPacketOutcomeSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketOutcomeHistoryCount, + AcceptedAlignedTightFollowUpPacketOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedBroadRecommendedPacketOutcomeSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketOutcomeHistoryCount, + AcceptedAlignedBroadRecommendedPacketOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedTightFollowUpPacketOutcomeTrustSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketTrustOutcomeHistoryCount, + AcceptedAlignedTightFollowUpPacketTrustOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsAcceptedAlignedBroadRecommendedPacketOutcomeTrustSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketTrustOutcomeHistoryCount, + AcceptedAlignedBroadRecommendedPacketTrustOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketOutcomeTrustSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketTrustReliableOutcomeHistoryCount, + AcceptedAlignedTightFollowUpPacketTrustReliableOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketOutcomeAcceptedHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketTrustReliableLaneOutcomeHistoryCount, + AcceptedAlignedTightFollowUpPacketTrustReliableLaneOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketOutcomeTrustSource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableOutcomeHistoryCount, + AcceptedAlignedBroadRecommendedPacketTrustReliableOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketOutcomeAcceptedHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableLaneOutcomeHistoryCount, + AcceptedAlignedBroadRecommendedPacketTrustReliableLaneOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactAcceptedHistorySource( OutcomeSummary.PreferredCoachHandoffSourceLabel) - || HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactOutcomeHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactAcceptedHistorySource( + || HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactOutcomeHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeHistoryCount, + AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactAcceptedHistorySource( OutcomeSummary.PreferredCoachHandoffSourceLabel) - || HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactOutcomeHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactOutcomeReliableHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeReliableHistoryCount += 1; - AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeReliableAccumulator += - OutcomeSummary.OutcomeScore; - } - if (HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactOutcomeReliableHistorySource( - OutcomeSummary.PreferredCoachHandoffSourceLabel)) - { - Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeReliableHistoryCount += 1; - AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeReliableAccumulator += - OutcomeSummary.OutcomeScore; - } + || HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactOutcomeHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeHistoryCount, + AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedTightFollowUpPacketExactOutcomeReliableHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeReliableHistoryCount, + AcceptedAlignedTightFollowUpPacketTrustReliableExactOutcomeReliableAccumulator, + OutcomeSummary.OutcomeScore + ); + AccumulateOutcomeHistory( + HyperTwistTrainingRepositoryLibraryInternal::IsReliableAcceptedAlignedBroadRecommendedPacketExactOutcomeReliableHistorySource( + OutcomeSummary.PreferredCoachHandoffSourceLabel), + Summary.AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeReliableHistoryCount, + AcceptedAlignedBroadRecommendedPacketTrustReliableExactOutcomeReliableAccumulator, + OutcomeSummary.OutcomeScore + ); int32 AcceptedAlignedTightReliableSuffixCount = INDEX_NONE; if (HyperTwistTrainingRepositoryLibraryInternal::TryGetAcceptedAlignedTightFollowUpPacketExactOutcomeReliableSuffixCount( OutcomeSummary.PreferredCoachHandoffSourceLabel, diff --git a/docs/ops/HYPERTWIST_REFACTORING_TOOLCHAIN_2026-06-22.md b/docs/ops/HYPERTWIST_REFACTORING_TOOLCHAIN_2026-06-22.md index 9e985eb..08e0d84 100644 --- a/docs/ops/HYPERTWIST_REFACTORING_TOOLCHAIN_2026-06-22.md +++ b/docs/ops/HYPERTWIST_REFACTORING_TOOLCHAIN_2026-06-22.md @@ -2134,3 +2134,26 @@ follow-up on `2026-06-30`: - the next worthwhile continuation still is another bounded recommendation-history or queue-summary body reduction seam rather than website churn, renderer widening, or XR scope creep + +Later same-day recommendation-history micro-follow-up on `2026-06-30`: + +- the next adjacent reduction stayed inside the same + `DeriveCoachRecommendationHistorySummary(...)` body: + - added a bounded local `AccumulateOutcomeHistory(...)` helper + - collapsed the repeated simple accepted-aligned history-count plus + outcome-accumulator branches into that helper without changing source truth +- the validation loop remained green again: + - `scripts/run-hypertwist-sentrux-source-only.sh` -> `Quality: 6245` + - `scripts/run-hypertwist-sentrux-gate.sh` -> `No degradation detected` + - `scripts/run-hypertwist-gitnexus-analyze.sh` -> fallback to + `npx gitnexus@latest`, `16,566` nodes, `39,157` edges, `680` clusters, + `300` flows + - `scripts/run-hypertwist-gitnexus-status.sh` -> `Indexed commit: 0e53f2b`, + `Current commit: 0e53f2b`, `Status: up-to-date` + - `scripts/run-hypertwist-remote-unreal-build.sh --worktree-root 'C:\htpp'` + -> `Target is up to date`, `Result: Succeeded`, UnrealBuildTool + `Total execution time: 8.53 seconds` +- next best move from this exact landed state: + - keep draining the same recommendation-history function, especially the + remaining lane-conditioned outcome branches near the top of that block, + before widening into unrelated website or XR work diff --git a/docs/v6_5_deep_manual_pack/HyperTwist/DEVELOPMENT.md b/docs/v6_5_deep_manual_pack/HyperTwist/DEVELOPMENT.md index 3bacc05..5f8d797 100644 --- a/docs/v6_5_deep_manual_pack/HyperTwist/DEVELOPMENT.md +++ b/docs/v6_5_deep_manual_pack/HyperTwist/DEVELOPMENT.md @@ -2953,3 +2953,26 @@ follow-up (`2026-06-30`) - the next worthwhile continuation still is another bounded recommendation-history or queue-summary body reduction seam rather than website churn, renderer widening, or XR scope creep + +Later same-day recommendation-history micro-follow-up (`2026-06-30`): + +- the next adjacent reduction stayed inside the same + `DeriveCoachRecommendationHistorySummary(...)` body: + - added a bounded local `AccumulateOutcomeHistory(...)` helper + - collapsed the repeated simple accepted-aligned history-count plus + outcome-accumulator branches into that helper without changing source truth +- the validation loop remained green again: + - `scripts/run-hypertwist-sentrux-source-only.sh` -> `Quality: 6245` + - `scripts/run-hypertwist-sentrux-gate.sh` -> `No degradation detected` + - `scripts/run-hypertwist-gitnexus-analyze.sh` -> fallback to + `npx gitnexus@latest`, `16,566` nodes, `39,157` edges, `680` clusters, + `300` flows + - `scripts/run-hypertwist-gitnexus-status.sh` -> `Indexed commit: 0e53f2b`, + `Current commit: 0e53f2b`, `Status: up-to-date` + - `scripts/run-hypertwist-remote-unreal-build.sh --worktree-root 'C:\htpp'` + -> `Target is up to date`, `Result: Succeeded`, UnrealBuildTool + `Total execution time: 8.53 seconds` +- next best move from this exact landed state: + - keep draining the same recommendation-history function, especially the + remaining lane-conditioned outcome branches near the top of that block, + before widening into unrelated website or XR work