Firefly-810: Merge PR #1106 from firefly-810-mask

Firefly-810: Fixed mask not loading correctly when zooming

Author: robyww

docs/release-notes.md

@@ -5,8 +5,10 @@
 - To Access current (unstable) development - (Version _next_, unreleased) use docker tag: `nightly`
 
 ## Version 2021.2
+- 2021.2.4 (July 2021)
+  - docker tag: `latest`, `release-2021.2`, `release-2021.2.4`
 - 2021.2.3 (June 2021)
-  - docker tag: `latest`, `release-2021.2`, `release-2021.2.3`
+  - docker tag: `release-2021.2.3`
 - 2021.2.2 (June 2021)
   - docker tag: `release-2021.2.2`
 - 2021.2.1 (May 2021)
@@ -48,6 +50,11 @@
   - Fixed: further refinement to error handling when retrieving TAP error documents
   - Fixed: Simbad name resolution issue ([Firefly-797](https://github.com/Caltech-IPAC/firefly/pull/1103))
   - Fixed: Mouse zoom not working correctly ([Firefly-803](https://github.com/Caltech-IPAC/firefly/pull/1103))
+- 2021.2.4
+  - Fixed: Image mask not zooming correctly ([Firefly-810](https://github.com/Caltech-IPAC/firefly/pull/1106))
+  - Fixed: HiPS select table _i_ link not working ([Firefly-819](https://github.com/Caltech-IPAC/firefly/pull/1106))
+  - Fixed: bias slider not working in 3 color mode ([PR](https://github.com/Caltech-IPAC/firefly/pull/1106))
+  - Fixed: boolean table filters not working ([Firefly-805](https://github.com/Caltech-IPAC/firefly/pull/1104))
 
 
 ##### _Pull Request in this release_
@@ -58,7 +65,7 @@
 
 ## Version 2021.1
 - 2021.1.0  (February 2021)
-  - docker tag: `latest`, `release-2021.1`, `release-2021.1.0`
+  - docker tag: `release-2021.1`, `release-2021.1.0`
   - original release
 
 

src/firefly/java/edu/caltech/ipac/firefly/data/ServerParams.java

@@ -169,6 +169,8 @@ public class ServerParams {
     public static final String LOGOUT = "CmdLogout";
     public static final String TILE_SIZE = "tileSize";
     public static final String BACK_TO_URL= "backToUrl";
+    public static final String MASK_DATA= "maskData";
+    public static final String MASK_BITS= "maskBits";
 
     //Workspaces
     public static final String WS_LIST = "wsList"; // Gets the list of content/files

src/firefly/java/edu/caltech/ipac/firefly/server/rpc/VisServerCommands.java

@@ -187,8 +187,10 @@ public static class ByteAryCmd extends ServerCommandAccess.HttpCommand {
         public void processRequest(HttpServletRequest req, HttpServletResponse res, SrvParam sp) throws Exception {
 
             PlotState state= sp.getState();
+            boolean mask= sp.getOptionalBoolean(ServerParams.MASK_DATA,false);
+            int maskBits= sp.getOptionalInt(ServerParams.MASK_BITS,0);
             int tileSize = sp.getRequiredInt(ServerParams.TILE_SIZE);
-            byte [] byte1D= VisServerOps.getByteStretchArray(state,tileSize);
+            byte [] byte1D= VisServerOps.getByteStretchArray(state,tileSize,mask,maskBits);
             res.setContentType("application/octet-stream");
 
             ByteBuffer byteBuf = ByteBuffer.wrap(byte1D);

src/firefly/java/edu/caltech/ipac/firefly/server/visualize/DirectStretchUtils.java

@@ -13,10 +13,14 @@
 import edu.caltech.ipac.visualize.plot.Histogram;
 import edu.caltech.ipac.visualize.plot.ImageData;
 import edu.caltech.ipac.visualize.plot.ImageHeader;
+import edu.caltech.ipac.visualize.plot.ImageMask;
 import edu.caltech.ipac.visualize.plot.RangeValues;
 import edu.caltech.ipac.visualize.plot.plotdata.FitsRead;
 import edu.caltech.ipac.visualize.plot.plotdata.RGBIntensity;
 
+import java.awt.*;
+import java.util.ArrayList;
+import java.util.List;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.TimeUnit;
@@ -38,7 +42,8 @@ public class DirectStretchUtils {
         return flipped;
     }
 
-    public static byte [] getStretchData(PlotState state,  ActiveFitsReadGroup frGroup, int tileSize) throws InterruptedException {
+    public static byte [] getStretchData(PlotState state,  ActiveFitsReadGroup frGroup, int tileSize, boolean mask, long maskBits)
+            throws InterruptedException {
 
         FitsRead fr= frGroup.getFitsRead(state.firstBand());
         int totWidth= fr.getNaxis1();
@@ -118,6 +123,41 @@ public class DirectStretchUtils {
                 }
             }
 
+        }
+        else if (mask) {
+            float [] float1d= fr.getRawFloatAry();
+            final float [] flip1d= flipFloatArray(float1d,totWidth,totHeight);
+            byte1d= new byte[flip1d.length];
+
+            List<ImageMask> masksList=  new ArrayList<ImageMask>();
+            for(int j= 0; (j<31); j++) {
+                if (((maskBits>>j) & 1) != 0) {
+                    masksList.add(new ImageMask(j,Color.RED));
+                }
+            }
+            ImageMask[] maskAry= masksList.toArray(new ImageMask[0]);
+
+
+            for(int i= 0; i<xPanels; i++) {
+                for(int j= 0; j<yPanels; j++) {
+                    int width= (i<xPanels-1) ? tileSize : ((totWidth-1) % tileSize + 1);
+                    int height= (j<yPanels-1) ? tileSize : ((totHeight-1) % tileSize + 1);
+                    idx= (i*yPanels) +j;
+                    imageDataAry[idx]= new ImageData( maskAry,
+                            state.getRangeValues(), tileSize*i,tileSize*j, width, height);
+                    ImageData im= imageDataAry[idx];
+
+                    executor.execute(() -> im.stretchMaskAndSave(flip1d,fr.getNaxis1()));
+                }
+            }
+            executor.shutdown();
+            normalTermination= executor.awaitTermination(600, TimeUnit.SECONDS);
+            for (ImageData imageData : imageDataAry) {
+                byte[] tmpByteAry = imageData.getSavedStandardStretch();
+                System.arraycopy(tmpByteAry, 0, byte1d, bPos, tmpByteAry.length);
+                bPos += tmpByteAry.length;
+            }
+
         }
         else {
             float [] float1d= fr.getRawFloatAry();

src/firefly/java/edu/caltech/ipac/firefly/server/visualize/VisServerOps.java

@@ -406,12 +406,12 @@ public static float[] getFloatDataArray(PlotState state, Band band) {
 
 
 
-    public static byte[] getByteStretchArray(PlotState state, int tileSize) {
+    public static byte[] getByteStretchArray(PlotState state, int tileSize, boolean mask, long maskBits) {
         try {
             long start = System.currentTimeMillis();
             ActiveCallCtx ctx = CtxControl.prepare(state);
             ActiveFitsReadGroup frGroup= ctx.getFitsReadGroup();
-            byte [] byte1d= DirectStretchUtils.getStretchData(state,frGroup,tileSize);
+            byte [] byte1d= DirectStretchUtils.getStretchData(state,frGroup,tileSize,mask, maskBits);
             long elapse = System.currentTimeMillis() - start;
             PlotServUtils.statsLog("byteAry",
                     "total-MB", (float)byte1d.length / StringUtils.MEG,

src/firefly/java/edu/caltech/ipac/firefly/server/visualize/hips/HiPSMasterList.java

@@ -195,7 +195,7 @@ private void setupMeta(DataGroup dg, boolean bMulti) {
         col.setWidth(4);
         col.setFilterable(false);
         col.setSortable(false);
-        col.setLinkInfos(Collections.singletonList(new LinkInfo(null, INFO_ICON_STUB, null, "link to HiPS properties", null, null, null)));
+        col.setLinkInfos(Collections.singletonList(new LinkInfo(null, INFO_ICON_STUB, "${Properties}", "link to HiPS properties", null, null, null)));
     }
 
     private static DataGroup createTableDataFromListEntry(List<HiPSMasterListEntry> hipsMaps) {

src/firefly/java/edu/caltech/ipac/visualize/plot/ImageData.java

@@ -266,10 +266,24 @@ public byte[] stretch8bit(final float [] float1d, final Header header, final His
         return byteAry;
     }
 
+    public byte[] stretchMask(final float [] float1d, final int naxis1) {
+        byte [] byteAry= new byte[this.width * this.height];
+        byte blank_pixel_value = (byte) 255;
+        int[] pixelhist = new int[256];
+
+        ImageStretch.stretchPixelsForMask(x, lastPixel, y, lastLine, naxis1,
+                blank_pixel_value, float1d, byteAry, pixelhist, imageMasks);
+        return byteAry;
+    }
+
     public void stretch8bitAndSave(final float [] float1d, final Header header, final Histogram histogram) {
         this.saveStandardStretch = stretch8bit(float1d,header,histogram);
     }
 
+    public void stretchMaskAndSave(final float [] float1d, final int naxis1) {
+        this.saveStandardStretch = stretchMask(float1d,naxis1);
+    }
+
     public byte[] getSavedStandardStretch() { return this.saveStandardStretch; }
 
     public void stretch3ColorAndSave(float [][] float1dAry, ImageHeader [] imHeadAry, Histogram[] histAry,

src/firefly/js/data/ServerParams.js

@@ -65,6 +65,8 @@ export const ServerParams = {
         SCROLL_Y: 'scrollY',
         ZOOM_FACTOR: 'zoomFactor',
         RANGE_VALUES: 'rangeValues',
+        MASK_DATA: 'maskData',
+        MASK_BITS: 'maskBits',
 
         EXT_TYPE: 'extType',
         IMAGE: 'image',

src/firefly/js/util/Color.js

@@ -45,7 +45,7 @@ function toRGBA(/* String */ color, /* Number */ alpha=1) {
     }
 }
 
-function toRGB(/* String */ color) {
+export function toRGB(/* String */ color) {
     if (color.startsWith('rgb')) {
         const intColor= toRGBA(color);
         intColor.length=3;

src/firefly/js/util/WebUtil.js

@@ -69,7 +69,7 @@ export const isOffscreenCanvas= (b) => getGlobalObj().OffscreenCanvas && (b inst
 
 export function createCanvas(width,height) {
     const global= getGlobalObj();
-    const c = global.document ? global.document.createElement('canvas') : new OffscreenCanvas(w,h);
+    const c = global.document ? global.document.createElement('canvas') : new OffscreenCanvas(width,height);
     c.width = width;
     c.height = height;
     return c;