fixed unnecessary dependency between visualization and event log

Author: VikiPeeva

src/org/processmining/placebasedlpmdiscovery/lpmdiscovery/LPMDiscovery.java

@@ -7,11 +7,12 @@
 public interface LPMDiscovery {
 
     static LPMDiscovery getInstance() {
-        return placeBased(PlacesProvider.getInstance(), 100);
+        return placeBased(PlacesProvider.getInstance(), 50);
     }
 
     static LPMDiscovery placeBased(PlacesProvider placesProvider, int placeLimit) {
         return new PlaceBasedLPMDiscovery(placesProvider, placeLimit);
     }
+
     LPMDiscoveryResult from(XLog log);
 }

src/org/processmining/placebasedlpmdiscovery/lpmdiscovery/PlaceBasedLPMDiscovery.java

@@ -8,10 +8,8 @@
 import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algorithms.parameters.PlaceBasedLPMDiscoveryParameters;
 import org.processmining.placebasedlpmdiscovery.model.Place;
 import org.processmining.placebasedlpmdiscovery.model.discovery.LPMDiscoveryResult;
-import org.processmining.placebasedlpmdiscovery.model.discovery.StandardLPMDiscoveryResult;
 import org.processmining.placebasedlpmdiscovery.model.logs.EventLog;
 import org.processmining.placebasedlpmdiscovery.model.logs.XLogWrapper;
-import org.processmining.placebasedlpmdiscovery.model.serializable.LPMResult;
 import org.processmining.placebasedlpmdiscovery.prom.PlacesProvider;
 
 import java.util.Set;
@@ -35,9 +33,11 @@ public LPMDiscoveryResult from(XLog log) {
 
         LPMDiscoveryAlgBuilder builder = Main.createDefaultBuilder(log, parameters);
         Set<Place> places = this.placesProvider.from(log);
+
         FPGrowthForPlacesLPMBuildingInput lpmBuildingInput = new FPGrowthForPlacesLPMBuildingInput(eventLog, places);
         StandardLPMDiscoveryInput discoveryInput =
                 new StandardLPMDiscoveryInput(eventLog, lpmBuildingInput);
-        return new LPMResult((StandardLPMDiscoveryResult) builder.build().run(discoveryInput, parameters));
+
+        return builder.build().run(discoveryInput, parameters);
     }
 }

src/org/processmining/placebasedlpmdiscovery/model/serializable/LPMResult.java

@@ -5,7 +5,6 @@
 import org.processmining.placebasedlpmdiscovery.main.LPMDiscoveryConfig;
 import org.processmining.placebasedlpmdiscovery.model.LocalProcessModel;
 import org.processmining.placebasedlpmdiscovery.model.discovery.LPMDiscoveryResult;
-import org.processmining.placebasedlpmdiscovery.model.discovery.StandardLPMDiscoveryResult;
 import org.processmining.placebasedlpmdiscovery.model.exporting.exporters.Exporter;
 
 import java.io.OutputStream;
@@ -25,7 +24,7 @@ public LPMResult() {
     }
 
     // QA: How does one convert from one child to another of a superclass?
-    public LPMResult(StandardLPMDiscoveryResult result) {
+    public LPMResult(LPMDiscoveryResult result) {
         super(result.getAllLPMs());
         this.additionalResults = result.getAdditionalResults();
     }

src/org/processmining/placebasedlpmdiscovery/prom/plugins/mining/LPMDiscoveryPlugin.java

@@ -11,9 +11,10 @@
 import org.processmining.models.graphbased.directed.petrinet.Petrinet;
 import org.processmining.placebasedlpmdiscovery.Main;
 import org.processmining.placebasedlpmdiscovery.lpmbuilding.inputs.FPGrowthForPlacesLPMBuildingInput;
+import org.processmining.placebasedlpmdiscovery.lpmdiscovery.LPMDiscovery;
+import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algbuilder.LPMDiscoveryAlgBuilder;
 import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algorithms.inputs.LPMDiscoveryInput;
 import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algorithms.inputs.StandardLPMDiscoveryInput;
-import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algbuilder.LPMDiscoveryAlgBuilder;
 import org.processmining.placebasedlpmdiscovery.lpmdiscovery.algorithms.parameters.PlaceBasedLPMDiscoveryParameters;
 import org.processmining.placebasedlpmdiscovery.model.discovery.LPMDiscoveryResult;
 import org.processmining.placebasedlpmdiscovery.model.logs.EventLog;
@@ -40,6 +41,22 @@
 )
 public class LPMDiscoveryPlugin {
 
+    @UITopiaVariant(
+            affiliation = "RWTH - PADS",
+            author = "Viki Peeva",
+            email = "[email protected]",
+            uiLabel = "Basic Local Process Models Discovery"
+    )
+    @PluginVariant(
+            variantLabel = "Basic Local Process Models Discovery",
+            requiredParameterLabels = {0}
+    )
+    public static LPMDiscoveryResult mineBasicLPMs(UIPluginContext context, XLog log) {
+        ContextKeeper.setUp(context);
+
+        return LPMDiscovery.getInstance().from(log);
+    }
+
     @UITopiaVariant(
             affiliation = "RWTH - PADS",
             author = "Viki Peeva",

src/org/processmining/placebasedlpmdiscovery/prom/plugins/visualization/visualizers/LPMDiscoveryResultVisualizer.java

@@ -8,6 +8,7 @@
 import org.processmining.framework.plugin.annotations.PluginVariant;
 import org.processmining.placebasedlpmdiscovery.lpmdiscovery.dependencyinjection.LPMDiscoveryResultGuiceModule;
 import org.processmining.placebasedlpmdiscovery.model.discovery.LPMDiscoveryResult;
+import org.processmining.placebasedlpmdiscovery.model.serializable.LPMResult;
 import org.processmining.placebasedlpmdiscovery.prom.dependencyinjection.PromGuiceModule;
 import org.processmining.placebasedlpmdiscovery.view.controllers.DefaultLPMDiscoveryResultViewController;
 
@@ -23,17 +24,24 @@ public class LPMDiscoveryResultVisualizer {
     @PluginVariant(requiredParameterLabels = {0})
     public JComponent visualize(UIPluginContext context, LPMDiscoveryResult result) {
 
-        Injector injector = Guice.createInjector(new LPMDiscoveryResultGuiceModule(result),
-                new PromGuiceModule(context));
+        if (result.getInput() != null && result.getInput().getLog() != null) {
+            Injector injector = Guice.createInjector(new LPMDiscoveryResultGuiceModule(result),
+                    new PromGuiceModule(context));
 
 //        DefaultLPMDiscoveryResultViewModel model = new DefaultLPMDiscoveryResultViewModel(result);
 //        DefaultLPMDiscoveryResultComponent view = new DefaultLPMDiscoveryResultComponent()
 
-        DefaultLPMDiscoveryResultViewController controller =
-                injector.getInstance(DefaultLPMDiscoveryResultViewController.class);
+            DefaultLPMDiscoveryResultViewController controller =
+                    injector.getInstance(DefaultLPMDiscoveryResultViewController.class);
 
-        controller.getView().display(controller.getModel());
+            controller.getView().display(controller.getModel());
 
-        return controller.getView();
+            return controller.getView();
+        }
+
+        // if no event log, complex operations are not possible
+        LPMResult lpmResult = new LPMResult(result);
+        LPMResultVisualizer visualizer = new LPMResultVisualizer();
+        return visualizer.visualize(context, lpmResult);
     }
 }