From 7f3b0d5297006df708f9169e473448ec6f85e211 Mon Sep 17 00:00:00 2001 From: Tuukka Lehtonen Date: Fri, 29 Nov 2019 23:41:42 +0200 Subject: [PATCH] Interim fix for orientation/location of static info on network branches Previously the profile style implementation would reset the edge info direction to match the simplified edge data whenever the style result was recalculated from the database. Now the style just tells the static info node the DistrictNetworkEdgeNode it is supposed to visualize and the info node will calculate the info pos/orientation based on that during render. This is an interim fix because the style implementation will be removed once this visualization is moved into the district visualization toolbox from the diagram profile implementation. At that point the visualization also needs to be optimized to only show the information for the edges inside the current viewbox. gitlab #47 Change-Id: I8b8ea78378df5749cf59405d7f9fae8d9848698e --- .../nodes/DistrictNetworkStaticInfoNode.java | 10 +- .../DistrictNetworkStaticInfoStyle.java | 118 ++++++++++-------- 2 files changed, 68 insertions(+), 60 deletions(-) diff --git a/org.simantics.district.network.ui/src/org/simantics/district/network/ui/nodes/DistrictNetworkStaticInfoNode.java b/org.simantics.district.network.ui/src/org/simantics/district/network/ui/nodes/DistrictNetworkStaticInfoNode.java index c8eae515..523f7e80 100644 --- a/org.simantics.district.network.ui/src/org/simantics/district/network/ui/nodes/DistrictNetworkStaticInfoNode.java +++ b/org.simantics.district.network.ui/src/org/simantics/district/network/ui/nodes/DistrictNetworkStaticInfoNode.java @@ -22,13 +22,11 @@ public class DistrictNetworkStaticInfoNode extends G2DNode implements DeferredNo private static final Font FONT = new Font(Font.SANS_SERIF, Font.PLAIN, DPIUtil.upscale(10)); - private static final Point2D UNIT_X = new Point2D.Double(1.0, 0.0); - public static final String NODE_KEY = "DISTRICT_NETWORK_STATIC_INFO"; String info = null; Point2D origin = new Point2D.Double(); - Point2D direction = UNIT_X; + Point2D direction = new Point2D.Double(); private DistrictNetworkEdgeNode edgeNode = null; @@ -103,8 +101,8 @@ public class DistrictNetworkStaticInfoNode extends G2DNode implements DeferredNo } public void setLocation(Point2D origin, Point2D direction) { - this.origin = origin; - this.direction = direction; + this.origin.setLocation(origin); + this.direction.setLocation(direction); } public void setInfo(String info) { @@ -113,5 +111,7 @@ public class DistrictNetworkStaticInfoNode extends G2DNode implements DeferredNo public void setEdgeNode(DistrictNetworkEdgeNode n) { this.edgeNode = n; + // Ensure that origin/location are recalculated + prevZoomLevel = -1; } } diff --git a/org.simantics.district.network.ui/src/org/simantics/district/network/ui/styles/DistrictNetworkStaticInfoStyle.java b/org.simantics.district.network.ui/src/org/simantics/district/network/ui/styles/DistrictNetworkStaticInfoStyle.java index 5c5c80b7..89e48813 100644 --- a/org.simantics.district.network.ui/src/org/simantics/district/network/ui/styles/DistrictNetworkStaticInfoStyle.java +++ b/org.simantics.district.network.ui/src/org/simantics/district/network/ui/styles/DistrictNetworkStaticInfoStyle.java @@ -1,6 +1,7 @@ package org.simantics.district.network.ui.styles; import java.awt.geom.Point2D; +import java.util.Objects; import java.util.Set; import org.simantics.Simantics; @@ -49,55 +50,76 @@ public class DistrictNetworkStaticInfoStyle extends StyleBase types = graph.getTypes(mapElement); + boolean isEdge = types.contains(DN.Edge); + boolean isVertex = types.contains(DN.Vertex); if (!isEdge && !isVertex) return null; - + if (isEdge) { Resource diagram = graph.getSingleObject(mapElement, Layer0.getInstance(graph).PartOf); Set edgesToUse = graph.syncRequest(new MidBranchEdgeSetRequest(diagram), TransientCacheListener.instance()); if (!edgesToUse.contains(mapElement)) return null; } - - DiagramResource DIA = DiagramResource.getInstance(graph); - StructuralResource2 STR = StructuralResource2.getInstance(graph); - + Resource module = DistrictNetworkUtil.getMappedComponentCached(graph, mapElement); if (module == null) return null; + StructuralResource2 STR = StructuralResource2.getInstance(graph); Resource moduleType = graph.getPossibleType(module, STR.Component); if (moduleType == null) return null; - + Function1 function = getUCPipelineInfoFunctionCached(graph, moduleType); if (function == null) return null; - + String result; try { Variable variable = Variables.getVariable(graph, module); @@ -109,76 +131,61 @@ public class DistrictNetworkStaticInfoStyle extends StyleBase"; + // the requested variable is missing from the UC + String message = e.getMessage(); + LOGGER.warn("Missing variable for calculating style with function {} {}", function, message); + result = "<" + message + ">"; } - + if (isVertex) { + DiagramResource DIA = DiagramResource.getInstance(graph); double[] coords = graph.getRelatedValue(mapElement, DIA.HasLocation); Point2D p = DistrictNetworkNodeUtils.calculatePoint2D(new Point2D.Double(coords[0], coords[1]), null); - return new StyleResult(p, p, result); - } - else if (isEdge) { - Resource v1 = graph.getSingleObject(mapElement, DN.HasStartVertex); - double[] coords1 = graph.getRelatedValue(v1, DIA.HasLocation); - Resource v2 = graph.getSingleObject(mapElement, DN.HasEndVertex); - double[] coords2 = graph.getRelatedValue(v2, DIA.HasLocation); - Point2D p1 = DistrictNetworkNodeUtils.calculatePoint2D(new Point2D.Double(coords1[0], coords1[1]), null); - Point2D p2 = DistrictNetworkNodeUtils.calculatePoint2D(new Point2D.Double(coords2[0], coords2[1]), null); - - return new StyleResult(p1, p2, result); + return new StyleResult(mapElement, p, result); + } else if (isEdge) { + return new StyleResult(mapElement, EDGE, result); } - return null; } - + @Override public void applyStyleForNode(EvaluationContext evaluationContext, INode parent, StyleResult result) { if (result == null) { cleanupStyleForNode(evaluationContext, parent); return; } - + ParentNode root = (ParentNode) NodeUtil.getNearestParentOfType(parent, RTreeNode.class); if (root != null) { DeferredRenderingNode deferred = ProfileVariables.claimChild(root, "", STATIC_INFO_DEFERRED, DeferredRenderingNode.class, evaluationContext); deferred.setZIndex(Integer.MAX_VALUE-1); } - + DistrictNetworkStaticInfoNode node = ProfileVariables.claimChild(parent, "*", DistrictNetworkStaticInfoNode.NODE_KEY, DistrictNetworkStaticInfoNode.class, evaluationContext); if (node == null) return; Point2D p1 = result.p1; - Point2D p2 = result.p2; - - if (p1.equals(p2)) { + if (p1 != EDGE) { node.setLocation(p1, new Point2D.Double(1.0, 0.0)); - } - else { - double sign = Math.signum(p1.getX() - p2.getX()); - Point2D.Double origin = new Point2D.Double(0.5 * (p1.getX() + p2.getX()), 0.5 * (p1.getY() + p2.getY())); - Point2D direction = new Point2D.Double(0.5 * sign * (p1.getX() - p2.getX()), 0.5 * sign * (p1.getY() - p2.getY())); - - node.setLocation(origin, direction); - + } else { + // StaticInfoNode takes care of positioning the text during render + // based on the found DistrictNetworkEdgeNode's information. for (IG2DNode n : ((ConnectionNode)parent).getNodes()) { if (n instanceof DistrictNetworkEdgeNode) { node.setEdgeNode((DistrictNetworkEdgeNode) n); } } } - + node.setInfo(result.info); } - + private static Function1 getUCPipelineInfoFunctionCached(ReadGraph graph, Resource componentType) throws DatabaseException { return graph.syncRequest(new UCPipelineInfoRequest(componentType), TransientCacheListener.instance()); } - + private static final class UCPipelineInfoRequest extends ResourceRead> { public UCPipelineInfoRequest(Resource resource) { super(resource); @@ -190,7 +197,7 @@ public class DistrictNetworkStaticInfoStyle extends StyleBase