Fix/placemark rotation and tilt

worldwind-examples/src/main/java/gov/nasa/worldwindx/PlacemarksDemoActivity.java

@@ -30,6 +30,8 @@
 import gov.nasa.worldwind.BasicWorldWindowController;
 import gov.nasa.worldwind.PickedObject;
 import gov.nasa.worldwind.PickedObjectList;
+import gov.nasa.worldwind.WorldWind;
+import gov.nasa.worldwind.geom.Offset;
 import gov.nasa.worldwind.geom.Position;
 import gov.nasa.worldwind.layer.RenderableLayer;
 import gov.nasa.worldwind.render.ImageSource;
@@ -140,9 +142,11 @@ public PlaceLevelOfDetailSelector(Resources resources, Place place) {
          * @param rc
          * @param placemark      The placemark needing a level of detail selection
          * @param cameraDistance The distance from the placemark to the camera (meters)
+         *
+         * @return if placemark should display or skip its rendering
          */
         @Override
-        public void selectLevelOfDetail(RenderContext rc, Placemark placemark, double cameraDistance) {
+        public boolean selectLevelOfDetail(RenderContext rc, Placemark placemark, double cameraDistance) {
 
             boolean highlighted = placemark.isHighlighted();
             boolean highlightChanged = this.lastHighlightState != highlighted;
@@ -211,7 +215,13 @@ public void selectLevelOfDetail(RenderContext rc, Placemark placemark, double ca
             this.lastHighlightState = highlighted;
 
             // Update the placemark's attributes bundle
-            placemark.setAttributes(this.attributes);
+            if (this.attributes != null) {
+                this.attributes.setDrawLabel(highlighted || cameraDistance <= LEVEL_1_DISTANCE);
+                placemark.setAttributes(this.attributes);
+                return true; // Placemark visible
+            } else {
+                return false; // Placemark invisible
+            }
         }
 
         protected static PlacemarkAttributes getPlacemarkAttributes(Resources resources, Place place) {
@@ -273,6 +283,7 @@ protected static PlacemarkAttributes createPlacemarkAttributes(Resources resourc
             //IconBitmapFactory factory = new IconBitmapFactory(resources, resourceId);
             //placemarkAttributes.setImageSource(ImageSource.fromBitmapFactory(factory)).setImageScale(scale);
             placemarkAttributes.setImageSource(ImageSource.fromResource(resourceId)).setImageScale(scale).setMinimumImageScale(0.5);
+            placemarkAttributes.getLabelAttributes().setTextOffset(new Offset(WorldWind.OFFSET_PIXELS, -24, WorldWind.OFFSET_FRACTION, 0.0));
             return placemarkAttributes;
         }
     }
@@ -560,6 +571,8 @@ private void createPlaceIcons() {
                 placemark.setLevelOfDetailSelector(new PlaceLevelOfDetailSelector(getResources(), place));
                 placemark.setEyeDistanceScaling(true);
                 placemark.setEyeDistanceScalingThreshold(PlaceLevelOfDetailSelector.LEVEL_1_DISTANCE);
+                placemark.setLabel(place.name);
+                placemark.setAltitudeMode(WorldWind.CLAMP_TO_GROUND);
 
                 // On a background thread, we can add Placemarks to a RenderableLayer that is
                 // NOT attached to the WorldWindow. If the layer was attached to the WorldWindow

worldwind-examples/src/main/java/gov/nasa/worldwindx/PlacemarksMilStd2525Activity.java

@@ -86,6 +86,7 @@ protected void onPostExecute(Void notUsed) {
             Placemark drone = new Placemark(
                 Position.fromDegrees(32.4520, 63.44553, 3000),
                 MilStd2525.getPlacemarkAttributes("SFAPMFQM--GIUSA", modifiers, null));
+            drone.getAttributes().setDrawLeader(true);
 
             symbolLayer.addRenderable(drone);
 
@@ -96,6 +97,7 @@ protected void onPostExecute(Void notUsed) {
             Placemark launcher = new Placemark(
                 Position.fromDegrees(32.4014, 63.3894, 0),
                 MilStd2525.getPlacemarkAttributes("SHGXUCFRMS----G", modifiers, null));
+            launcher.setAltitudeMode(WorldWind.CLAMP_TO_GROUND);
 
             symbolLayer.addRenderable(launcher);
 
@@ -109,6 +111,7 @@ protected void onPostExecute(Void notUsed) {
             Placemark machineGun = new Placemark(
                 Position.fromDegrees(32.3902, 63.4161, 0),
                 MilStd2525.getPlacemarkAttributes("SFGPEWRH--MTUSG", modifiers, null));
+            machineGun.setAltitudeMode(WorldWind.CLAMP_TO_GROUND);
 
             symbolLayer.addRenderable(machineGun);
 

worldwind-examples/src/main/java/gov/nasa/worldwindx/milstd2525/MilStd2525.java

@@ -269,7 +269,8 @@ public Bitmap createBitmap() {
             Rect bounds = imageInfo.getImageBounds();       // The extents of the image, including text modifiers
             this.placemarkOffset = new Offset(
                 WorldWind.OFFSET_PIXELS, centerPoint.x, // x offset
-                WorldWind.OFFSET_PIXELS, bounds.height() - centerPoint.y); // y offset converted to lower-left origin
+                // Use billboarding or lollipopping to prevent icon clipping by terrain as described in MIL-STD-2525C APPENDIX F.5.1.1.2
+                WorldWind.OFFSET_PIXELS, 0/*bounds.height() - centerPoint.y*/); // y offset converted to lower-left origin
 
             // Apply the placemark offset to the attributes on the main thread. This is necessary to synchronize write
             // access to placemarkAttributes from the thread that invokes this BitmapFactory and read access from the

worldwind-examples/src/main/java/gov/nasa/worldwindx/milstd2525/MilStd2525LevelOfDetailSelector.java

@@ -59,9 +59,11 @@ public static void setNearThreshold(double nearThreshold) {
      * @param rc             The current render contents
      * @param placemark      The placemark needing a level of detail selection
      * @param cameraDistance The distance from the placemark to the camera (meters)
+     *
+     * @return if placemark should display or skip its rendering
      */
     @Override
-    public void selectLevelOfDetail(RenderContext rc, Placemark placemark, double cameraDistance) {
+    public boolean selectLevelOfDetail(RenderContext rc, Placemark placemark, double cameraDistance) {
         if (!(placemark instanceof MilStd2525Placemark)) {
             throw new IllegalArgumentException(
                 Logger.logMessage(Logger.ERROR, "MilStd2525LevelOfDetailSelector", "selectLevelOfDetail",
@@ -108,5 +110,7 @@ public void selectLevelOfDetail(RenderContext rc, Placemark placemark, double ca
         if (this.placemarkAttributes != null) {
             milStdPlacemark.setAttributes(this.placemarkAttributes);
         }
+
+        return true; // Placemark is always visible
     }
 }

worldwind/src/main/java/gov/nasa/worldwind/render/RenderContext.java

@@ -432,7 +432,7 @@ public Texture getText(String text, TextAttributes attributes) {
     }
 
     public Texture renderText(String text, TextAttributes attributes) {
-        TextCacheKey key = new TextCacheKey().set(text, attributes);
+        TextCacheKey key = this.scratchTextCacheKey.set(text, attributes);
         Texture texture = null;
 
         if (text != null && attributes != null) {

worldwind/src/main/java/gov/nasa/worldwind/shape/Label.java

@@ -445,10 +445,11 @@ protected void makeDrawable(RenderContext rc) {
         // origin at the text's bottom-left corner and axes that extend up and to the right from the origin point.
         int w = texture.getWidth();
         int h = texture.getHeight();
+        double s = this.activeAttributes.scale;
         this.activeAttributes.textOffset.offsetForSize(w, h, renderData.offset);
         renderData.unitSquareTransform.setTranslation(
-            renderData.screenPlacePoint.x - renderData.offset.x,
-            renderData.screenPlacePoint.y - renderData.offset.y,
+            renderData.screenPlacePoint.x - renderData.offset.x * s,
+            renderData.screenPlacePoint.y - renderData.offset.y * s,
             renderData.screenPlacePoint.z);
 
         // Apply the label's rotation according to its rotation value and orientation mode. The rotation is applied
@@ -462,7 +463,7 @@ protected void makeDrawable(RenderContext rc) {
         }
 
         // Apply the label's translation and scale according to its text size.
-        renderData.unitSquareTransform.multiplyByScale(w, h, 1);
+        renderData.unitSquareTransform.multiplyByScale(w * s, h * s, 1);
 
         WWMath.boundingRectForUnitSquare(renderData.unitSquareTransform, renderData.screenBounds);
         if (!rc.frustum.intersectsViewport(renderData.screenBounds)) {