bulltrackers-module 1.0.137 → 1.0.138

package/functions/computation-system/helpers/computation_pass_runner.js

@@ -26,11 +26,52 @@ async function runComputationPass(config, dependencies, computationManifest) {
     const yesterday = new Date(); yesterday.setUTCDate(yesterday.getUTCDate()-1); 
     const endDateUTC = new Date(Date.UTC(yesterday.getUTCFullYear(), yesterday.getUTCMonth(), yesterday.getUTCDate()));
     const earliestDates = await getEarliestDataDates(config, dependencies);
-    const firstDate = earliestDates.absoluteEarliest; 
-    const startDateUTC = firstDate ? new Date(Date.UTC(firstDate.getUTCFullYear(), firstDate.getUTCMonth(), firstDate.getUTCDate())) : new Date(config.earliestComputationDate+'T00:00:00Z');
-    const allExpectedDates = getExpectedDateStrings(startDateUTC, endDateUTC);
+
+    // --- REMOVE OLD/REPLACE WITH NEW LOGIC FOR PROBLEM 3 & 4 ---
+    // const firstDate = earliestDates.absoluteEarliest; // <-- REMOVE
+    // const startDateUTC = firstDate ? new Date(Date.UTC(firstDate.getUTCFullYear(), firstDate.getUTCMonth(), firstDate.getUTCDate())) : new Date(config.earliestComputationDate+'T00:00:00Z'); // <-- REMOVE
+
     const passes = groupByPass(computationManifest);
     const calcsInThisPass = passes[passToRun] || []; if (!calcsInThisPass.length) return logger.log('WARN', `[PassRunner] No calcs for Pass ${passToRun}. Exiting.`);
+    
+    // --- START: NEW LOGIC FOR PROBLEM 3 & 4 ---
+    // Determine the earliest start date based *only* on data required for THIS pass.
+    const requiredRootData = new Set();
+    calcsInThisPass.forEach(c => {
+        (c.rootDataDependencies || []).forEach(dep => requiredRootData.add(dep));
+    });
+
+    let earliestStartDateForPass = null;
+    if (requiredRootData.size > 0) {
+        // Find the LATEST of the earliest dates for all required data types.
+        // e.g., If portfolio starts 09-25 and social starts 10-30, a calc
+        // needing both must start on 10-30.
+        let latestOfEarliestDates = new Date(0); // Start at epoch
+        requiredRootData.forEach(dep => {
+            const earliestDateForDep = earliestDates[dep]; // e.g., earliestDates.portfolio
+            if (earliestDateForDep && earliestDateForDep > latestOfEarliestDates) {
+                latestOfEarliestDates = earliestDateForDep;
+            }
+        });
+        
+        if (latestOfEarliestDates.getTime() > 0) {
+             earliestStartDateForPass = latestOfEarliestDates;
+        }
+    }
+
+    // Use the pass-specific date. Fall back to absolute earliest, then config.
+    const firstDate = earliestStartDateForPass || earliestDates.absoluteEarliest;
+    const startDateUTC = firstDate 
+        ? new Date(Date.UTC(firstDate.getUTCFullYear(), firstDate.getUTCMonth(), firstDate.getUTCDate())) 
+        : new Date(config.earliestComputationDate+'T00:00:00Z');
+    
+    logger.log('INFO', `[PassRunner] Pass ${passToRun} requires data: [${Array.from(requiredRootData).join(', ')}].`);
+    logger.log('INFO', `[PassRunner] Determined start date for this pass: ${startDateUTC.toISOString().slice(0, 10)}`);
+    // --- END: NEW LOGIC ---
+
+    const allExpectedDates = getExpectedDateStrings(startDateUTC, endDateUTC);
+    const firstDayOfBackfill = allExpectedDates.length > 0 ? allExpectedDates[0] : null; // --- MOVED FROM ABOVE ---
+    
     const standardCalcs = calcsInThisPass.filter(c => c.type==='standard'); 
     const metaCalcs = calcsInThisPass.filter(c => c.type==='meta');
     const processDate = async (dateStr) => {
@@ -39,7 +80,7 @@ async function runComputationPass(config, dependencies, computationManifest) {
             const rootData = await checkRootDataAvailability(dateStr, config, dependencies, earliestDates);
             if (!rootData) { logger.log('WARN', `[PassRunner] Skipping ${dateStr} for Pass ${passToRun}: No root data.`);return;}
             const existingResults = await fetchExistingResults(dateStr, calcsInThisPass, computationManifest, config, dependencies);
-            const { standardCalcsToRun, metaCalcsToRun } = filterCalculations(standardCalcs, metaCalcs, rootData.status, existingResults, passToRun, dateStr, logger);
+            const { standardCalcsToRun, metaCalcsToRun } = filterCalculations(standardCalcs, metaCalcs, rootData.status, existingResults, passToRun, dateStr, logger,dateStr === firstDayOfBackfill );            
             if (standardCalcsToRun.length === 0 && metaCalcsToRun.length === 0) {logger.log('INFO', `[PassRunner] All calcs for ${dateStr} Pass ${passToRun} are already complete. Skipping.`);return;}
             if (standardCalcsToRun.length) await runStandardComputationPass(dateToProcess, standardCalcsToRun, `Pass ${passToRun} (Standard)`, config, dependencies, rootData);
             if (metaCalcsToRun.length) await runMetaComputationPass(dateToProcess, metaCalcsToRun, `Pass ${passToRun} (Meta)`, config, dependencies, existingResults, rootData);

package/functions/computation-system/helpers/orchestration_helpers.js

@@ -72,20 +72,26 @@ async function fetchExistingResults(dateStr, calcsInPass, fullManifest, config,
 
 /** --- MODIFIED: Stage 5: Filter calculations to skip completed work ---
  */
-function filterCalculations(standardCalcs, metaCalcs, rootDataStatus, existingResults, passToRun, dateStr, logger) {
+function filterCalculations(standardCalcs, metaCalcs, rootDataStatus, existingResults, passToRun, dateStr, logger, isFirstDayOfBackfill = false) { 
     const skipped = new Set();
-    // Filter Standard Calcs
     const standardCalcsToRun = standardCalcs.filter(c => {
         if (existingResults[c.name]) {logger.log('TRACE', `[Pass ${passToRun}] Skipping ${c.name} for ${dateStr}. Result already exists.`);return false;}
-        const { canRun, missing } = checkRootDependencies(c, rootDataStatus);
-        if (canRun) return true;
-        logger.log('INFO', `[Pass ${passToRun}] Skipping ${c.name} for ${dateStr}. Missing root data: [${missing.join(', ')}]`);
-        skipped.add(c.name);
-        return false;
+        if (isFirstDayOfBackfill && c.isHistorical) {
+            logger.log('INFO', `[Pass ${passToRun}] Skipping ${c.name} for ${dateStr}. Historical calc on the first day of backfill (no yesterday).`);
+            skipped.add(c.name);
+            return false;
+        }
     });
     // Filter Meta Calcs
     const metaCalcsToRun = metaCalcs.filter(c => {
         if (existingResults[c.name]) {logger.log('TRACE', `[Pass ${passToRun} Meta] Skipping ${c.name} for ${dateStr}. Result already exists.`);skipped.add(c.name);return false;}
+        // --- START: RECOMMENDED ADDITION ---
+        if (isFirstDayOfBackfill && c.isHistorical) {
+            logger.log('INFO', `[Pass ${passToRun} Meta] Skipping ${c.name} for ${dateStr}. Historical calc on the first day of backfill (no yesterday).`);
+            skipped.add(c.name);
+            return false;
+        }
+        // --- END: RECOMMENDED ADDITION ---
         // 1. Check root data
         const { canRun, missing: missingRoot } = checkRootDependencies(c, rootDataStatus);
         if (!canRun) {logger.log('INFO', `[Pass ${passToRun} Meta] Skipping ${c.name} for ${dateStr}. Missing root data: [${missingRoot.join(', ')}]`);skipped.add(c.name);return false;}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bulltrackers-module",
-  "version": "1.0.137",
+  "version": "1.0.138",
   "description": "Helper Functions for Bulltrackers.",
   "main": "index.js",
   "files": [