Uploaded image for project: 'Snowstorm'
  1. Snowstorm
  2. STORM-1793

1) Treat all mini-map altitudes above 1020 m as the same height 2) Improve z-level accuracy

    XMLWordPrintableJSON

Details

    • Defect
    • Status: Closed
    • Minor
    • Resolution: Released
    • None
    • Active
    • None
    • Current beta - version doesn't matter
    • Code Review
    • Hide

      This change requires a corresponding server change which is now fully deployed to the main grid.

      You will need 2 avatars for this test, avatar A and B, both wearing some kind of flight feather, or some alternative way to reach high altitudes.

      With A go to a busy area, the junction of Dore with Ahern is a good place

      Open A's mini-map

      Cam around closely, verifying that the residents you see on the screen also show in the same place on the mini-map

      Hover your mouse over some of these dots: an inspector should appear. Verify that the name in the inspector belongs to the avatar you are looking at in the viewer

      Put A and B near each other, so A can see B's indicator on the mini-map
      With A set Preferences->Graphics draw distance to 128

      With A fly to 1000m

      With B fly up to 980m: A's mini-map shows a down-caret

      With B fly up to 1000m: A's mini-map shows a round dot

      With B fly up to 1010m: A's mini-map shows an up-caret

      With B fly up to 1030m: A's mini-map shows an up-caret

      With A fly up to 1030m: A'm mini-map shows a round dot

      With B fly up to 1500m: A's mini-map shows an X

      Right click on the mini-map and pick Zoom Close, Zoom Medium, and Zoom Far (or use the mouse wheel to zoom in and out): the X icon is readable at all zoom levels

      Show
      This change requires a corresponding server change which is now fully deployed to the main grid. You will need 2 avatars for this test, avatar A and B, both wearing some kind of flight feather, or some alternative way to reach high altitudes. With A go to a busy area, the junction of Dore with Ahern is a good place Open A's mini-map Cam around closely, verifying that the residents you see on the screen also show in the same place on the mini-map Hover your mouse over some of these dots: an inspector should appear. Verify that the name in the inspector belongs to the avatar you are looking at in the viewer Put A and B near each other, so A can see B's indicator on the mini-map With A set Preferences->Graphics draw distance to 128 With A fly to 1000m With B fly up to 980m: A's mini-map shows a down-caret With B fly up to 1000m: A's mini-map shows a round dot With B fly up to 1010m: A's mini-map shows an up-caret With B fly up to 1030m: A's mini-map shows an up-caret With A fly up to 1030m: A'm mini-map shows a round dot With B fly up to 1500m: A's mini-map shows an X Right click on the mini-map and pick Zoom Close, Zoom Medium, and Zoom Far (or use the mouse wheel to zoom in and out): the X icon is readable at all zoom levels

    Description

      The SL simulator has recently been fixed so that the CoarseLocationUpdate message properly returns 255 for all altitudes above 1020 meters.

      The code for the mini-map, in drawing the agent locations for equal, above or below needs to take this into account. It currently does not.

      LLNetMap::globalPosToView() computes a relative position:

      LLVector3d relative_pos_global = global_pos - gAgentCamera.getCameraPositionGlobal();

      The Z value needs to take the global camera pos, and slam anything above 1020 to be 1020, and all our problems will be solved.

      The routine that returns who is nearby can be enhanced to scan the character list and use that position data. This gives an accurate z-level value, even when above 1020m.

      In the case where the relative Z value between you and another avatar is unknown display an X on the mini-map.

      Attachments

        1. ZoomClose.png
          ZoomClose.png
          62 kB
        2. ZoomMedium.png
          ZoomMedium.png
          93 kB
        3. Firestorm_Version_wo_Serverchange.png
          Firestorm_Version_wo_Serverchange.png
          34 kB
        4. _thumb_57736.png
          _thumb_57736.png
          49 kB

        Issue Links

          Activity

            People

              Unassigned Unassigned
              simon.linden Simon Linden
              jira-users
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: