Image widget 'limits_from_pv'

Author: kasemir

app/display/model/src/main/java/org/csstudio/display/builder/model/widgets/plots/ImageWidget.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2015-2018 Oak Ridge National Laboratory.
+ * Copyright (c) 2015-2022 Oak Ridge National Laboratory.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -13,6 +13,7 @@
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propFile;
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propForegroundColor;
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propInteractive;
+import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propLimitsFromPV;
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propMaximum;
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.propMinimum;
 import static org.csstudio.display.builder.model.properties.CommonWidgetProperties.runtimePropConfigure;
@@ -409,6 +410,7 @@ public boolean configureFromXML(final ModelReader model_reader, final Widget wid
     private volatile WidgetProperty<ColorMap> data_colormap;
     private volatile ColorBarProperty color_bar;
     private volatile AxisWidgetProperty x_axis, y_axis;
+    private volatile WidgetProperty<Boolean> limits_from_pv;
     private volatile WidgetProperty<Integer> data_width, data_height;
     private volatile WidgetProperty<InterpolationType> data_interpolation;
     private volatile WidgetProperty<VImageType> data_color_mode;
@@ -440,6 +442,7 @@ protected void defineProperties(final List<WidgetProperty<?>> properties)
         properties.add(color_bar = new ColorBarProperty(this));
         properties.add(x_axis = new XAxisWidgetProperty(this));
         properties.add(y_axis = new YAxisWidgetProperty(this));
+        properties.add(limits_from_pv = propLimitsFromPV.createProperty(this, false)); // 'true' would be preferred but breaks existing displays
         properties.add(data_width = propDataWidth.createProperty(this, 100));
         properties.add(data_height = propDataHeight.createProperty(this, 100));
         properties.add(data_interpolation = propInterpolationType.createProperty(this, InterpolationType.AUTOMATIC));
@@ -531,6 +534,12 @@ public AxisWidgetProperty propYAxis()
         return y_axis;
     }
 
+    /** @return 'limits_from_pv' property */
+    public WidgetProperty<Boolean> propLimitsFromPV()
+    {
+        return limits_from_pv;
+    }
+
     /** @return 'data_width' property */
     public WidgetProperty<Integer> propDataWidth()
     {

app/display/model/src/main/resources/examples/plots_image.bob

@@ -274,33 +274,38 @@ the value range.</text>
     </rois>
   </widget>
   <widget type="label" version="2.0.0">
-    <name>Label_8</name>
-    <text>For PVAccess, a PV that serves an NTNDArray will provide
-both the image data and the width &amp; height information.
-(This is currently limited to byte[] data.
- For NTNDArray with short, int, etc., you need to read
- pva://that_pv/value to get just the data array,
- and configure the data width and height).
-
-For PVAccess or Channel Access, the data can be a waveform
-that contains the pixels values starting with the top-left pixel,
-followed by the pixels of the top row, then the second row
-and so on.
+    <class>SECTION</class>
+    <text>PV, Data</text>
+    <x>201</x>
+    <y>891</y>
+    <width>171</width>
+    <font use_class="true">
+      <font name="Default Bold" family="Liberation Sans" style="BOLD" size="14.0">
+      </font>
+    </font>
+    <foreground_color use_class="true">
+      <color name="Text" red="0" green="0" blue="0">
+      </color>
+    </foreground_color>
+    <transparent use_class="true">true</transparent>
+  </widget>
+  <widget type="label" version="2.0.0">
+    <text>The pixel data is an array that starts with the top-left pixel, followed by the pixels of the top row, then the second row and so on.
+    
+For PVAccess, a PV that serves an NTNDArray will provide not only the pixel data but also the width, height, signed/unsigned information and color mode.
 
-The image data width and height need to be configured for
-waveform PVs.
-For NTNDArray PVs, the image size is provided by the PV.</text>
+For Channel Access, the waveform PV only provides pixels. The data width, height, signedness and color mode need to be configured via properties of the image widget.
+</text>
     <x>201</x>
     <y>921</y>
     <width>421</width>
-    <height>279</height>
+    <height>209</height>
   </widget>
   <widget type="label" version="2.0.0">
-    <name>Label_9</name>
     <class>SECTION</class>
-    <text>PV, Data</text>
+    <text>Axes</text>
     <x>201</x>
-    <y>891</y>
+    <y>1130</y>
     <width>171</width>
     <font use_class="true">
       <font name="Default Bold" family="Liberation Sans" style="BOLD" size="14.0">
@@ -312,6 +317,17 @@ For NTNDArray PVs, the image size is provided by the PV.</text>
     </foreground_color>
     <transparent use_class="true">true</transparent>
   </widget>
+  <widget type="label" version="2.0.0">
+    <text>When the "Limits from PV" property of the widget is set, the X axis will range from 0 to the width of the image in pixels and the Y axis from 0 to the height. The origin will be placed in the top-left corner.
+
+When "Limits from PV" is not set, the range of the X and Y axes can be configured independently from the image size. This allows for example a calibration of pixel positions in millimeters.
+The axis direction can be inverted by swapping the axis minimum and maximum limit.
+</text>
+    <x>201</x>
+    <y>1150</y>
+    <width>421</width>
+    <height>209</height>
+  </widget>
   <widget type="label" version="2.0.0">
     <name>Label_10</name>
     <text>Region of Interest</text>

app/display/representation-javafx/src/main/java/org/csstudio/display/builder/representation/javafx/widgets/plots/ImageRepresentation.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2015-2020 Oak Ridge National Laboratory.
+ * Copyright (c) 2015-2022 Oak Ridge National Laboratory.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -424,13 +424,19 @@ private void contentChanged(final WidgetProperty<?> property, final Object old_v
     {
         final VType value = model_widget.runtimePropValue().getValue();
         if (value instanceof VNumberArray)
-            image_plot.setValue(model_widget.propDataWidth().getValue(),
-                                model_widget.propDataHeight().getValue(),
+        {
+            // Can't get size from PV, use data_width, .._height
+            final int width = model_widget.propDataWidth().getValue();
+            final int height = model_widget.propDataHeight().getValue();
+            if (model_widget.propLimitsFromPV().getValue())
+                image_plot.setAxisRange(0.0, width, height, 0.0);
+            image_plot.setValue(width, height,
                                 ((VNumberArray) value).getData(),
                                 model_widget.propDataUnsigned().getValue(),
                                 model_widget.propDataColorMode().getValue());
+        }
         else if (value instanceof VImage)
-        {
+        {   // Use VImage metadata
             final VImage image = (VImage) value;
             boolean isUnsigned;
             switch (image.getDataType())
@@ -444,7 +450,12 @@ else if (value instanceof VImage)
             	default:
             		isUnsigned = false;
             }
-            image_plot.setValue(image.getWidth(), image.getHeight(), image.getData(),
+
+            final int width = image.getWidth();
+            final int height = image.getHeight();
+            if (model_widget.propLimitsFromPV().getValue())
+                image_plot.setAxisRange(0.0, width, height, 0.0);
+            image_plot.setValue(width, height, image.getData(),
                                 isUnsigned, image.getVImageType());
         }
         else if (value != null)