Merge pull request #357 from Caltech-IPAC/DM-10149-PeriodogramIssues

DM-10149 fix the phase value range of phase folded chart

Author: cwang2016

src/firefly/html/firefly.html

@@ -21,7 +21,7 @@
                 options : {
                     MenuItemKeys: {maskOverlay:true},
                     catalogSpacialOp: 'polygonWhenPlotExist',
-                    charts: {chartEngine: ''}
+                    charts: {chartEngine: 'plotly'}
                 }
             }
         };

src/firefly/js/templates/lightcurve/LcPeriodPlotly.jsx

@@ -340,8 +340,8 @@ class PhaseFoldingChart extends Component {
                     //color: 'blue',
                     color: 'rgba(63, 127, 191, 0.5)'
                 },
-                x: data.map((d) => d[0]),
-                y: data.map((d) => d[1]),
+                x: data&&data.map((d) => d[0]),
+                y: data&&data.map((d) => d[1]),
                 hoverinfo: 'text'
             }],
             plotlyLayout: {

src/firefly/js/templates/lightcurve/LcPhaseTable.js

@@ -75,7 +75,7 @@ export function doPFCalculate(flux, time, period, tzero) {
  */
 export function getPhase(time, timeZero, period,  dec=DEC_PHASE) {
     var q = (time - timeZero)/period;
-    var p = q >= 0  ? (q - Math.floor(q)) : (q + Math.floor(-q));
+    var p = q >= 0  ? (q - Math.floor(q)) : (q + Math.floor(-q) + 1.0);
 
     return p.toFixed(dec);
 }