Merge pull request #40 from carlolm/master

add: input date ranges into summary stats

Author: carlolm

client/src/components/SummaryStats.js

@@ -30,7 +30,7 @@ class SummaryStats extends Component {
       console.log(files[0].name);
       const fileName = files[0].name;
       CSV.parseCSV(reader.result).then((strategyData) => {
-        const summaryStats = calcSummaryStats(strategyData);
+        const summaryStats = calcSummaryStats(strategyData, '1/1/00', '9/1/17');
         this.setState({ strategyData, summaryStats, fileName });
       });
     };

client/src/utils/dataAnalysis.js

@@ -133,11 +133,11 @@ const calcSummaryStats = (priceData, startDate, endDate) => {
   let asset = priceData;
 
   if (startDate && endDate) {
-    console.log(startDate, endDate);
-    asset = asset.filter(data => data.date >= new Date(startDate) && data.date <= new Date(endDate));
+    asset = asset.filter(data => new Date(data.date) >= (new Date(startDate)) && new Date(data.date) <= (new Date(endDate)));
   }
 
   // Calculate Overall Return
+  console.log(asset);
   const first = asset[0];
   const last = asset[asset.length - 1];
   console.log('******', first, last);