From 6d61bbd0f3e76ed0aaba3343c6b52059643e8b68 Mon Sep 17 00:00:00 2001
From: Bhabesh <bhabesh.mukhopadhyay@nibio.no>
Date: Wed, 10 Jun 2020 11:52:20 +0200
Subject: [PATCH] Java Logging

Simple java logging system implement
---
 .../alternariamodel/AlternariaModel.java      | 39 ++++++++++--------
 .../model/alternariamodel/DataMatrix.java     |  5 +++
 .../alternariamodel/AlternariaModelTest.java  | 40 ++++++++++---------
 3 files changed, 50 insertions(+), 34 deletions(-)

diff --git a/src/main/java/no/nibio/vips/model/alternariamodel/AlternariaModel.java b/src/main/java/no/nibio/vips/model/alternariamodel/AlternariaModel.java
index 2bce9ab..3cfa194 100644
--- a/src/main/java/no/nibio/vips/model/alternariamodel/AlternariaModel.java
+++ b/src/main/java/no/nibio/vips/model/alternariamodel/AlternariaModel.java
@@ -48,6 +48,8 @@ import java.util.Collections;
 import java.util.Date;
 import java.util.List;
 import java.util.TimeZone;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 import no.nibio.vips.entity.ModelConfiguration;
 import no.nibio.vips.entity.Result;
 import no.nibio.vips.entity.ResultImpl;
@@ -62,12 +64,13 @@ import no.nibio.vips.util.JSONUtil;
 import no.nibio.vips.util.ModelUtil;
 import no.nibio.vips.util.WeatherUtil;
 
+
 /**
  *
  * @author bhabesh
  */
 public class AlternariaModel extends I18nImpl implements Model{
-  
+    public final static Logger      LOGGER              =   Logger.getLogger(AlternariaModel.class.getName());
     public final static String      NAME_MODEL_ID       =   "ALTERNARIA";
     public final static ModelId     MODEL_ID            =   new ModelId(NAME_MODEL_ID);
     public final static int         THRESHOLD_LW        =   30;                         // Threshold for leave wetness
@@ -93,6 +96,7 @@ public class AlternariaModel extends I18nImpl implements Model{
     @Override
     public List<Result> getResult() throws ModelExcecutionException
     {
+        
         /**
          * method name : getResult
          * @param      :
@@ -119,7 +123,12 @@ public class AlternariaModel extends I18nImpl implements Model{
         {
             Result              result              = new ResultImpl();
             
-                                
+                                if(null == dataMatrix.getParamStringValueForDate(currentDate, DataMatrix.SPRAYING_DATE))
+                                {
+                                    // DO Nothing -- Consider same accumulatedDSV
+                                }
+                                else
+                                {
                                     if(dataMatrix.getParamStringValueForDate(currentDate, DataMatrix.SPRAYING_DATE).equals(YES))
                                     {
                                         accumulatedDSV  = 0;
@@ -129,24 +138,28 @@ public class AlternariaModel extends I18nImpl implements Model{
                                     {
                                         accumulatedDSV      =   accumulatedDSV  +   dataMatrix.getParamIntValueForDate(currentDate, DataMatrix.DAILY_DISEASE_SEVERITY_VALUE);
                                     }
+                                }
                                     
                                 
                     result.setValidTimeStart(currentDate);
                     result.setWarningStatus(getWarningStatus(accumulatedDSV));
-                    
+
                     result.setValue(CommonNamespaces.NS_WEATHER, DataMatrix.TEMPERATURE_MEAN, dFormat.format(this.dataMatrix.getParamValueForDate(currentDate, DataMatrix.TEMPERATURE_MEAN)));
                     result.setValue(NAME_MODEL_ID, DataMatrix.WET_HOUR, iFormat.format(this.dataMatrix.getParamValueForDate(currentDate, DataMatrix.LEAF_WETNESS_DURATION)));
                     result.setValue(NAME_MODEL_ID, DataMatrix.DAILY_DISEASE_SEVERITY_VALUE_SUM, iFormat.format(accumulatedDSV));
                     result.setValue(NAME_MODEL_ID, DataMatrix.DAILY_DISEASE_SEVERITY_VALUE, iFormat.format(this.dataMatrix.getParamValueForDate(currentDate, DataMatrix.DAILY_DISEASE_SEVERITY_VALUE)));
-                   
+
+                    result.setValue(NAME_MODEL_ID, DataMatrix.THRESHOLD_DSV_BASE, String.valueOf(THRESHOLD_DSV_BASE));
+                    result.setValue(NAME_MODEL_ID, DataMatrix.THRESHOLD_DSV_MAX, String.valueOf(THRESHOLD_DSV_MIN));
+                    
                     results.add(result);
                     
             cal.setTime(currentDate);
             cal.add(Calendar.DATE, 1);
             currentDate = cal.getTime();               
         }
-    //    System.out.println("DataMatrix : "+dataMatrix);
-         //System.out.println("-----------------------------------------------------------------");
+
+        //LOGGER.log(Level.INFO, "DataMatrix-Value 03: "+dataMatrix);
         return results;
     }
 
@@ -389,9 +402,7 @@ public class AlternariaModel extends I18nImpl implements Model{
         WeatherUtil wUtil = new WeatherUtil();
                 // Setting timezone
         this.timeZone = TimeZone.getTimeZone((String) config.getConfigParameter("timeZone"));
-        //System.out.println("TimeZone=" + this.timeZone);
-                            
-                            
+
                                     sprayingDates                                   =   (null == mapper.convertValue(config.getConfigParameter(DataMatrix.SPRAYING_DATES), new TypeReference<List<Date>>(){})) 
                                                                                         ?   null
                                                                                         :   mapper.convertValue(config.getConfigParameter(DataMatrix.SPRAYING_DATES), new TypeReference<List<Date>>(){});
@@ -404,7 +415,7 @@ public class AlternariaModel extends I18nImpl implements Model{
         {
                 
                 weatherObj.setTimeMeasured(wUtil.pragmaticAdjustmentToMidnight(weatherObj.getTimeMeasured(), timeZone));
-                //System.out.println(" weatherObj : "+weatherObj);
+                
                 Date    sprayDate   =   null;
                 
                 switch(weatherObj.getElementMeasurementTypeId())
@@ -538,12 +549,8 @@ public class AlternariaModel extends I18nImpl implements Model{
             dateHourlyLw_previousDay = dateHourlyLw_currentDay;
             
         }
- /*
-        Gson gson   = new Gson();
-        System.out.println("Data matrix in JSON : "+gson.toJson(dataMatrix));
-   */     
-                
-    //    System.out.println("Data matrix : "+dataMatrix ); 
+   
+          //LOGGER.log(Level.INFO, dataMatrix.toString());
     }
 
   
diff --git a/src/main/java/no/nibio/vips/model/alternariamodel/DataMatrix.java b/src/main/java/no/nibio/vips/model/alternariamodel/DataMatrix.java
index c7789b8..6b5588f 100644
--- a/src/main/java/no/nibio/vips/model/alternariamodel/DataMatrix.java
+++ b/src/main/java/no/nibio/vips/model/alternariamodel/DataMatrix.java
@@ -52,5 +52,10 @@ public class DataMatrix extends DateMap{
     public final static String  SPRAYING_DATES                  =   "sprayingDates";//  Spraying dates
     public final static String  SPRAYING_DATE                   =   "sprayingDate"; //   Spray Date
     
+    public final static String  THRESHOLD_DSV_BASE              =   "THRESHOLD_DSV_BASE"; //   THRESHOLD_DSV_BASE
+    public final static String  THRESHOLD_DSV_MAX               =   "THRESHOLD_DSV_MAX"; //   THRESHOLD_DSV_MAX
+
+    
+    
     
 }
diff --git a/src/test/java/no/nibio/vips/model/alternariamodel/AlternariaModelTest.java b/src/test/java/no/nibio/vips/model/alternariamodel/AlternariaModelTest.java
index d3e3fe6..4e160a3 100644
--- a/src/test/java/no/nibio/vips/model/alternariamodel/AlternariaModelTest.java
+++ b/src/test/java/no/nibio/vips/model/alternariamodel/AlternariaModelTest.java
@@ -61,7 +61,7 @@ public class AlternariaModelTest {
     private     final   String      CONST_TEST_DATA_06              =   "TEST_DATA_06";
     private     final   String      CONST_TEST_DATA_07              =   "TEST_DATA_07";
     
-    private     final   String      EFFECTED_FILE_WEATHER_TEST_DATA =   CONST_TEST_DATA_06;
+    private     final   String      EFFECTED_FILE_WEATHER_TEST_DATA =   CONST_TEST_DATA_04;
     
     public AlternariaModelTest() {
     }
@@ -89,17 +89,19 @@ public class AlternariaModelTest {
     public void testGetResult() throws Exception {
         System.out.println("getResult");
         ModelConfiguration config   = this.getConfiguration(getWeatherDataFile());
-        config.setConfigParameter("sprayingDates", this.getConfigurationSprayingDates(getResetDataFile()));
+        config.setConfigParameter("sprayingDates", this.getConfigurationSprayingDates(getResetDataFile())); 
         AlternariaModel instance = new AlternariaModel();
         instance.setConfiguration(config);
         List<Result> result = instance.getResult();
         assertNotNull(result);
        
-        
+       /* 
         for(Result res:result)
         {
+            // LOGGER.log(Level.INFO, res.toString());
             System.out.println(res.toString());
         }
+       */ 
         
         
     }
@@ -283,7 +285,7 @@ public class AlternariaModelTest {
             config.setModelId(AlternariaModel.MODEL_ID.toString());
             
             
-            config.setConfigParameter("timeZone", "Europe/Helsinki");
+            config.setConfigParameter("timeZone", "Europe/Oslo");
             BufferedInputStream inputStream = new BufferedInputStream(this.getClass().getResourceAsStream(fileName));
             JsonFactory f = new MappingJsonFactory();
             JsonParser jp = f.createParser(inputStream);
@@ -354,26 +356,28 @@ public class AlternariaModelTest {
 
             Date firstDate = null;
             Date lastDate = null;
-            if(all.isArray())
+            if(null != all)
             {
-                for(JsonNode node : all){
-                   
-                    Date timeMeasuredForSpray = (Date)mapper.convertValue(node, new TypeReference<Date>(){});
-                    //System.out.println("Spraying Date : "+timeMeasuredForSpray);
-                    if(timeMeasuredForSpray != null )
-                    {
-                        sprayingDates.add(timeMeasuredForSpray);
+                if(all.isArray())
+                {
+                    for(JsonNode node : all){
+
+                        Date timeMeasuredForSpray = (Date)mapper.convertValue(node, new TypeReference<Date>(){});
+                        //System.out.println("Spraying Date : "+timeMeasuredForSpray);
+                        if(timeMeasuredForSpray != null )
+                        {
+                            sprayingDates.add(timeMeasuredForSpray);
+                        }
+
                     }
 
                 }
-
-            }
-            else
-            {
-                fail("Data input from file is not a JSON array for list of spraying dates");
+                else
+                {
+                    fail("Data input from file is not a JSON array for list of spraying dates");
+                }
             }
 
-
             
             return sprayingDates;
         } catch (IOException ex) {
-- 
GitLab