From f3cb25065f518b9569404aef7373a36c24072ef3 Mon Sep 17 00:00:00 2001 From: Sam Gondelman Date: Tue, 14 Jul 2015 17:36:22 -0700 Subject: [PATCH 01/14] can't move when hydras at base, hands can deactivate individually --- interface/src/avatar/SkeletonModel.cpp | 16 ++++++++++++---- interface/src/devices/SixenseManager.cpp | 17 +++++++++-------- 2 files changed, 21 insertions(+), 12 deletions(-) diff --git a/interface/src/avatar/SkeletonModel.cpp b/interface/src/avatar/SkeletonModel.cpp index 20d458195d..0eb54b7c07 100644 --- a/interface/src/avatar/SkeletonModel.cpp +++ b/interface/src/avatar/SkeletonModel.cpp @@ -120,8 +120,8 @@ void SkeletonModel::simulate(float deltaTime, bool fullUpdate) { Hand* hand = _owningAvatar->getHand(); hand->getLeftRightPalmIndices(leftPalmIndex, rightPalmIndex); - const float HAND_RESTORATION_RATE = 0.25f; - if (leftPalmIndex == -1 || rightPalmIndex == -1) { + const float HAND_RESTORATION_RATE = 0.25f; + if (leftPalmIndex == -1 && rightPalmIndex == -1) { // palms are not yet set, use mouse if (_owningAvatar->getHandState() == HAND_STATE_NULL) { restoreRightHandPosition(HAND_RESTORATION_RATE, PALM_PRIORITY); @@ -138,8 +138,16 @@ void SkeletonModel::simulate(float deltaTime, bool fullUpdate) { restoreLeftHandPosition(HAND_RESTORATION_RATE, PALM_PRIORITY); } else { - applyPalmData(geometry.leftHandJointIndex, hand->getPalms()[leftPalmIndex]); - applyPalmData(geometry.rightHandJointIndex, hand->getPalms()[rightPalmIndex]); + if (leftPalmIndex != -1) { + applyPalmData(geometry.leftHandJointIndex, hand->getPalms()[leftPalmIndex]); + } else { + restoreLeftHandPosition(HAND_RESTORATION_RATE, PALM_PRIORITY); + } + if (rightPalmIndex != -1) { + applyPalmData(geometry.rightHandJointIndex, hand->getPalms()[rightPalmIndex]); + } else { + restoreRightHandPosition(HAND_RESTORATION_RATE, PALM_PRIORITY); + } } if (_isFirstPerson) { diff --git a/interface/src/devices/SixenseManager.cpp b/interface/src/devices/SixenseManager.cpp index 12b6f4263b..c82027cb38 100644 --- a/interface/src/devices/SixenseManager.cpp +++ b/interface/src/devices/SixenseManager.cpp @@ -244,14 +244,6 @@ void SixenseManager::update(float deltaTime) { palm->setTrigger(data->trigger); palm->setJoystick(data->joystick_x, data->joystick_y); - handleButtonEvent(data->buttons, numActiveControllers - 1); - handleAxisEvent(data->joystick_x, data->joystick_y, data->trigger, numActiveControllers - 1); - - // Emulate the mouse so we can use scripts - if (Menu::getInstance()->isOptionChecked(MenuOption::SixenseMouseInput) && !_controllersAtBase) { - emulateMouse(palm, numActiveControllers - 1); - } - // NOTE: Sixense API returns pos data in millimeters but we IMMEDIATELY convert to meters. glm::vec3 position(data->pos[0], data->pos[1], data->pos[2]); position *= METERS_PER_MILLIMETER; @@ -260,6 +252,15 @@ void SixenseManager::update(float deltaTime) { const float CONTROLLER_AT_BASE_DISTANCE = 0.075f; if (glm::length(position) < CONTROLLER_AT_BASE_DISTANCE) { numControllersAtBase++; + palm->setActive(false); + } else { + handleButtonEvent(data->buttons, numActiveControllers - 1); + handleAxisEvent(data->joystick_x, data->joystick_y, data->trigger, numActiveControllers - 1); + + // Emulate the mouse so we can use scripts + if (Menu::getInstance()->isOptionChecked(MenuOption::SixenseMouseInput) && !_controllersAtBase) { + emulateMouse(palm, numActiveControllers - 1); + } } // Transform the measured position into body frame. From 6a0df442f98658cd9728f3a0a5ece4e2d5b772c7 Mon Sep 17 00:00:00 2001 From: Sam Gondelman Date: Tue, 14 Jul 2015 17:59:59 -0700 Subject: [PATCH 02/14] emulated mouse events don't get sent back to the keyboard (can't right click and drag with sixense, etc) --- interface/src/Application.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/interface/src/Application.cpp b/interface/src/Application.cpp index eabeb27d5e..79bdcea634 100644 --- a/interface/src/Application.cpp +++ b/interface/src/Application.cpp @@ -1568,7 +1568,9 @@ void Application::mouseMoveEvent(QMouseEvent* event, unsigned int deviceID) { return; } - _keyboardMouseDevice.mouseMoveEvent(event, deviceID); + if (deviceID == 0) { + _keyboardMouseDevice.mouseMoveEvent(event, deviceID); + } } @@ -1589,7 +1591,9 @@ void Application::mousePressEvent(QMouseEvent* event, unsigned int deviceID) { if (activeWindow() == _window) { - _keyboardMouseDevice.mousePressEvent(event); + if (deviceID == 0) { + _keyboardMouseDevice.mousePressEvent(event); + } if (event->button() == Qt::LeftButton) { _mouseDragStarted = getTrueMouse(); @@ -1629,7 +1633,9 @@ void Application::mouseReleaseEvent(QMouseEvent* event, unsigned int deviceID) { } if (activeWindow() == _window) { - _keyboardMouseDevice.mouseReleaseEvent(event); + if (deviceID == 0) { + _keyboardMouseDevice.mouseReleaseEvent(event); + } if (event->button() == Qt::LeftButton) { _mousePressed = false; From 316c322118ac3be6aaadfb2ed4465d52f5935e85 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 09:36:51 -0700 Subject: [PATCH 03/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 9 ++++++--- cmake/modules/FindPolyVox.cmake | 3 ++- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index cfaf7ed293..85387856a8 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -48,12 +48,15 @@ endif () if (WIN32) - set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY ${INSTALL_DIR}/PolyVoxCore/lib/PolyVoxCore.lib CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/PolyVoxCore/lib/Debug/PolyVoxCore.lib CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/PolyVoxCore/lib/Release/PolyVoxCore.lib CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/PolyVoxUtil/lib/PolyVoxUtil.lib CACHE FILEPATH "polyvox util library") elseif (APPLE) -set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") +set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") +set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxUtil.dylib CACHE FILEPATH "polyvox util library") else () - set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxUtil.so CACHE FILEPATH "polyvox util library") endif () diff --git a/cmake/modules/FindPolyVox.cmake b/cmake/modules/FindPolyVox.cmake index 60a639e87c..18ba9b6671 100644 --- a/cmake/modules/FindPolyVox.cmake +++ b/cmake/modules/FindPolyVox.cmake @@ -24,7 +24,8 @@ hifi_library_search_hints("polyvox") find_path(POLYVOX_CORE_INCLUDE_DIRS PolyVoxCore/SimpleVolume.h PATH_SUFFIXES include include/PolyVoxCore HINTS ${POLYVOX_SEARCH_DIRS}) # find_path(POLYVOX_UTIL_INCLUDE_DIRS PolyVoxUtil/Serialization.h PATH_SUFFIXES include include/PolyVoxUtil HINTS ${POLYVOX_SEARCH_DIRS}) -find_library(POLYVOX_CORE_LIBRARY NAMES PolyVoxCore PATH_SUFFIXES lib HINTS ${POLYVOX_SEARCH_DIRS}) +find_library(POLYVOX_CORE_LIBRARY_DEBUG NAMES PolyVoxCore PATH_SUFFIXES lib/Debug HINTS ${POLYVOX_SEARCH_DIRS}) +find_library(POLYVOX_CORE_LIBRARY_RELEASE NAMES PolyVoxCore PATH_SUFFIXES lib/Release lib HINTS ${POLYVOX_SEARCH_DIRS}) # find_library(POLYVOX_UTIL_LIBRARY NAMES PolyVoxUtil PATH_SUFFIXES lib HINTS ${POLYVOX_SEARCH_DIRS}) From fd84893c612de9496d0f4a9c5ae1984359bbd3b6 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 09:42:26 -0700 Subject: [PATCH 04/14] build separate debug and release versions of polyvox library --- cmake/modules/FindPolyVox.cmake | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cmake/modules/FindPolyVox.cmake b/cmake/modules/FindPolyVox.cmake index 18ba9b6671..08e9f1c6a2 100644 --- a/cmake/modules/FindPolyVox.cmake +++ b/cmake/modules/FindPolyVox.cmake @@ -28,6 +28,8 @@ find_library(POLYVOX_CORE_LIBRARY_DEBUG NAMES PolyVoxCore PATH_SUFFIXES lib/Debu find_library(POLYVOX_CORE_LIBRARY_RELEASE NAMES PolyVoxCore PATH_SUFFIXES lib/Release lib HINTS ${POLYVOX_SEARCH_DIRS}) # find_library(POLYVOX_UTIL_LIBRARY NAMES PolyVoxUtil PATH_SUFFIXES lib HINTS ${POLYVOX_SEARCH_DIRS}) +include(SelectLibraryConfigurations) +select_library_configurations(POLYVOX_CORE) # if (WIN32) # find_path(POLYVOX_DLL_PATH polyvox.dll PATH_SUFFIXES bin HINTS ${POLYVOX_SEARCH_DIRS}) From 443cfc8970e642bcfd969b37bec148f23697c75f Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 10:31:21 -0700 Subject: [PATCH 05/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index 85387856a8..fcb8e8cf04 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -3,7 +3,7 @@ set(EXTERNAL_NAME polyvox) include(ExternalProject) ExternalProject_Add( ${EXTERNAL_NAME} - URL http://hifi-public.s3.amazonaws.com/dependencies/polyvox.zip + URL http://hifi-public.s3.amazonaws.com/dependencies/polyvox-master-2015-7-15.zip URL_MD5 904b840328278c9b36fa7a14be730c34 CMAKE_ARGS -DENABLE_EXAMPLES=OFF -DCMAKE_INSTALL_PREFIX:PATH= BINARY_DIR ${EXTERNAL_PROJECT_PREFIX}/build From a71c5cae5fd54d5df39748f72519140639a1a133 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 10:33:22 -0700 Subject: [PATCH 06/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index fcb8e8cf04..9e08a1018e 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -4,7 +4,7 @@ include(ExternalProject) ExternalProject_Add( ${EXTERNAL_NAME} URL http://hifi-public.s3.amazonaws.com/dependencies/polyvox-master-2015-7-15.zip - URL_MD5 904b840328278c9b36fa7a14be730c34 + URL_MD5 d7965dc206e6f9bbdb210a9612f56382 CMAKE_ARGS -DENABLE_EXAMPLES=OFF -DCMAKE_INSTALL_PREFIX:PATH= BINARY_DIR ${EXTERNAL_PROJECT_PREFIX}/build LOG_DOWNLOAD 1 From 3bc182730a77487ef567ef25e188049cb8003e34 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 10:52:38 -0700 Subject: [PATCH 07/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index 9e08a1018e..3cc26f72bc 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -4,7 +4,7 @@ include(ExternalProject) ExternalProject_Add( ${EXTERNAL_NAME} URL http://hifi-public.s3.amazonaws.com/dependencies/polyvox-master-2015-7-15.zip - URL_MD5 d7965dc206e6f9bbdb210a9612f56382 + URL_MD5 9268bc5dc4141b303521dd67c133f1df CMAKE_ARGS -DENABLE_EXAMPLES=OFF -DCMAKE_INSTALL_PREFIX:PATH= BINARY_DIR ${EXTERNAL_PROJECT_PREFIX}/build LOG_DOWNLOAD 1 From c24c6ad39f2e80f52d3312c26b590be60ae1ea27 Mon Sep 17 00:00:00 2001 From: Thijs Wenker Date: Wed, 15 Jul 2015 19:56:01 +0200 Subject: [PATCH 08/14] Fixes chess game (pieces were dieing as soon as you dropped them) --- examples/entityScripts/chessPiece.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/examples/entityScripts/chessPiece.js b/examples/entityScripts/chessPiece.js index b6dfca9efd..24ba7b33e8 100644 --- a/examples/entityScripts/chessPiece.js +++ b/examples/entityScripts/chessPiece.js @@ -130,10 +130,10 @@ var others = Entities.findEntities(this.properties.position, this.properties.dimensions.y); for (var i = 0; i < others.length; i++) { - var piece = others[i]; + var pieceID = others[i]; - if (piece.id != this.entityID) { - var properties = Entities.getEntityProperties(piece); + if (pieceID != this.entityID) { + var properties = Entities.getEntityProperties(pieceID); var isWhite = properties.modelURL.search("White") !== -1; var type = (properties.modelURL.search("King") !== -1) ? 4 : @@ -147,7 +147,7 @@ if (myPos.i === piecePos.i && myPos.j === piecePos.j && type !== -2) { var position = this.getAbsolutePosition((isWhite) ? { i: type, j: -1 } : { i: 7 - type, j: 8 }, properties.dimensions.y / 2.0); - Entities.editEntity(piece, { + Entities.editEntity(pieceID, { position: position }); break; From a444d6a2a26abef25959b2f392ee3e280f03028f Mon Sep 17 00:00:00 2001 From: Andrew Meadows Date: Wed, 15 Jul 2015 13:19:59 -0700 Subject: [PATCH 09/14] remove hungarian notation --- examples/grab.js | 424 ++++++++++++++++++++++++++--------------------- 1 file changed, 235 insertions(+), 189 deletions(-) diff --git a/examples/grab.js b/examples/grab.js index 13c91bebb8..05bcf128e2 100644 --- a/examples/grab.js +++ b/examples/grab.js @@ -10,97 +10,13 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -var MOVE_TIMESCALE = 0.1; -var INV_MOVE_TIMESCALE = 1.0 / MOVE_TIMESCALE; var MAX_SOLID_ANGLE = 0.01; // objects that appear smaller than this can't be grabbed -var CLOSE_ENOUGH = 0.001; -var ZERO_VEC3 = { x: 0, y: 0, z: 0 }; -var ANGULAR_DAMPING_RATE = 0.40; - -// NOTE: to improve readability global variable names start with 'g' -var gIsGrabbing = false; -var gGrabbedEntity = null; -var gActionID = null; -var gEntityProperties; -var gStartPosition; -var gStartRotation; -var gCurrentPosition; -var gOriginalGravity = ZERO_VEC3; -var gPlaneNormal = ZERO_VEC3; - -// gMaxGrabDistance is a function of the size of the object. -var gMaxGrabDistance; - -// gGrabMode defines the degrees of freedom of the grab target positions -// relative to gGrabStartPosition options include: -// xzPlane (default) -// verticalCylinder (SHIFT) -// rotate (CONTROL) -// Modes to eventually support?: -// xyPlane -// yzPlane -// polar -// elevationAzimuth -var gGrabMode = "xzplane"; - -// gGrabOffset allows the user to grab an object off-center. It points from the object's center -// to the point where the ray intersects the grab plane (at the moment the grab is initiated). -// Future target positions of the ray intersection are on the same plane, and the offset is subtracted -// to compute the target position of the object's center. -var gGrabOffset = { x: 0, y: 0, z: 0 }; - -var gTargetPosition; -var gTargetRotation; -var gLiftKey = false; // SHIFT -var gRotateKey = false; // CONTROL - -var gInitialMouse = { x: 0, y: 0 }; -var gPreviousMouse = { x: 0, y: 0 }; -var gMouseCursorLocation = { x: 0, y: 0 }; -var gMouseAtRotateStart = { x: 0, y: 0 }; - -var gBeaconHeight = 0.10; - -// var gAngularVelocity = ZERO_VEC3; - -// TODO: play sounds again when we aren't leaking AudioInjector threads -// var grabSound = SoundCache.getSound("https://hifi-public.s3.amazonaws.com/eric/sounds/CloseClamp.wav"); -// var releaseSound = SoundCache.getSound("https://hifi-public.s3.amazonaws.com/eric/sounds/ReleaseClamp.wav"); -// var VOLUME = 0.0; - -var gBeaconHeight = 0.10; -var BEACON_COLOR = { - red: 200, - green: 200, - blue: 200 -}; -var BEACON_WIDTH = 2; +var ZERO_VEC3 = {x: 0, y: 0, z: 0}; +var IDENTITY_QUAT = {x: 0, y: 0, z: 0, w: 0}; -var gBeacon = Overlays.addOverlay("line3d", { - color: BEACON_COLOR, - alpha: 1, - visible: false, - lineWidth: BEACON_WIDTH -}); - -function updateDropLine(position) { - Overlays.editOverlay(gBeacon, { - visible: true, - start: { - x: position.x, - y: position.y + gBeaconHeight, - z: position.z - }, - end: { - x: position.x, - y: position.y - gBeaconHeight, - z: position.z - } - }); -} - -function mouseIntersectionWithPlane(pointOnPlane, planeNormal, event) { +// helper function +function mouseIntersectionWithPlane(pointOnPlane, planeNormal, event, maxDistance) { var cameraPosition = Camera.getPosition(); var localPointOnPlane = Vec3.subtract(pointOnPlane, cameraPosition); var distanceFromPlane = Vec3.dot(localPointOnPlane, planeNormal); @@ -117,7 +33,7 @@ function mouseIntersectionWithPlane(pointOnPlane, planeNormal, event) { var useMaxForwardGrab = false; if (Math.abs(dirDotNorm) > MIN_RAY_PLANE_DOT) { var distanceToIntersection = distanceFromPlane / dirDotNorm; - if (distanceToIntersection > 0 && distanceToIntersection < gMaxGrabDistance) { + if (distanceToIntersection > 0 && distanceToIntersection < maxDistance) { // ray points into the plane localIntersection = Vec3.multiply(pickRay.direction, distanceFromPlane / dirDotNorm); } else { @@ -134,52 +50,160 @@ function mouseIntersectionWithPlane(pointOnPlane, planeNormal, event) { // we re-route the intersection to be in front at max distance. var rayDirection = Vec3.subtract(pickRay.direction, Vec3.multiply(planeNormal, dirDotNorm)); rayDirection = Vec3.normalize(rayDirection); - localIntersection = Vec3.multiply(rayDirection, gMaxGrabDistance); + localIntersection = Vec3.multiply(rayDirection, maxDistance); localIntersection = Vec3.sum(localIntersection, Vec3.multiply(planeNormal, distanceFromPlane)); } var worldIntersection = Vec3.sum(cameraPosition, localIntersection); return worldIntersection; } -function computeNewGrabPlane() { - if (!gIsGrabbing) { +// Mouse class stores mouse click and drag info +Mouse = function() { + this.current = {x: 0, y: 0 }; + this.previous = {x: 0, y: 0 }; + this.rotateStart = {x: 0, y: 0 }; + this.cursorRestore = {x: 0, y: 0}; +} + +Mouse.prototype.startDrag = function(position) { + this.current = {x: position.x, y: position.y}; + this.startRotateDrag(); +} + +Mouse.prototype.updateDrag = function(position) { + this.current = {x: position.x, y: position.y }; +} + +Mouse.prototype.startRotateDrag = function() { + this.previous = {x: this.current.x, y: this.current.y}; + this.rotateStart = {x: this.current.x, y: this.current.y}; + this.cursorRestore = { x: Window.getCursorPositionX(), y: Window.getCursorPositionY() }; +} + +Mouse.prototype.getDrag = function() { + var delta = {x: this.current.x - this.previous.x, y: this.current.y - this.previous.y}; + this.previous = {x: this.current.x, y: this.current.y}; + return delta; +} + +Mouse.prototype.restoreRotateCursor = function() { + Window.setCursorPosition(this.cursorRestore.x, this.cursorRestore.y); + this.current = {x: this.rotateStart.x, y: this.rotateStart.y}; +} + +var mouse = new Mouse(); + + +// Beacon class stores info for drawing a line at object's target position +Beacon = function() { + this.height = 0.10; + this.overlayID = Overlays.addOverlay("line3d", { + color: {red: 200, green: 200, blue: 200}, + alpha: 1, + visible: false, + lineWidth: 2 + }); +} + +Beacon.prototype.enable = function() { + Overlays.editOverlay(this.overlayID, { visible: true }); +} + +Beacon.prototype.disable = function() { + Overlays.editOverlay(this.overlayID, { visible: false }); +} + +Beacon.prototype.updatePosition = function(position) { + Overlays.editOverlay(this.overlayID, { + visible: true, + start: { + x: position.x, + y: position.y + this.height, + z: position.z + }, + end: { + x: position.x, + y: position.y - this.height, + z: position.z + } + }); +} + +var beacon = new Beacon(); + + +// TODO: play sounds again when we aren't leaking AudioInjector threads +// var grabSound = SoundCache.getSound("https://hifi-public.s3.amazonaws.com/eric/sounds/CloseClamp.wav"); +// var releaseSound = SoundCache.getSound("https://hifi-public.s3.amazonaws.com/eric/sounds/ReleaseClamp.wav"); +// var VOLUME = 0.0; + + +// Grabber class stores and computes info for grab behavior +Grabber = function() { + this.isGrabbing = false; + this.entityID = null; + this.actionID = null; + this.startPosition = ZERO_VEC3; + this.lastRotation = IDENTITY_QUAT; + this.currentPosition = ZERO_VEC3; + this.planeNormal = ZERO_VEC3; + + this.originalGravity = ZERO_VEC3; + // maxDistance is a function of the size of the object. + this.maxDistance; + + // mode defines the degrees of freedom of the grab target positions + // relative to startPosition options include: + // xzPlane (default) + // verticalCylinder (SHIFT) + // rotate (CONTROL) + this.mode = "xzplane"; + + // offset allows the user to grab an object off-center. It points from the object's center + // to the point where the ray intersects the grab plane (at the moment the grab is initiated). + // Future target positions of the ray intersection are on the same plane, and the offset is subtracted + // to compute the target position of the object's center. + this.offset = {x: 0, y: 0, z: 0 }; + + this.targetPosition; + this.targetRotation; + + this.liftKey = false; // SHIFT + this.rotateKey = false; // CONTROL +} + +Grabber.prototype.computeNewGrabPlane = function() { + if (!this.isGrabbing) { return; } - var maybeResetMousePosition = false; - if (gGrabMode !== "rotate") { - gMouseAtRotateStart = gMouseCursorLocation; + var modeWasRotate = (this.mode == "rotate"); + this.mode = "xzPlane"; + this.planeNormal = {x: 0, y: 1, z: 0 }; + if (this.rotateKey) { + this.mode = "rotate"; + mouse.startRotateDrag(); } else { - maybeResetMousePosition = true; - } - gGrabMode = "xzPlane"; - gPlaneNormal = { x: 0, y: 1, z: 0 }; - if (gLiftKey) { - if (!gRotateKey) { - gGrabMode = "verticalCylinder"; - // a new planeNormal will be computed each move + if (modeWasRotate) { + // we reset the mouse screen position whenever we stop rotating + mouse.restoreRotateCursor(); } - } else if (gRotateKey) { - gGrabMode = "rotate"; - } + if (this.liftKey) { + this.mode = "verticalCylinder"; + // NOTE: during verticalCylinder mode a new planeNormal will be computed each move + } + } - gPointOnPlane = Vec3.sum(gCurrentPosition, gGrabOffset); - var xzOffset = Vec3.subtract(gPointOnPlane, Camera.getPosition()); + this.pointOnPlane = Vec3.sum(this.currentPosition, this.offset); + var xzOffset = Vec3.subtract(this.pointOnPlane, Camera.getPosition()); xzOffset.y = 0; - gXzDistanceToGrab = Vec3.length(xzOffset); - - if (gGrabMode !== "rotate" && maybeResetMousePosition) { - // we reset the mouse position whenever we stop rotating - Window.setCursorPosition(gMouseAtRotateStart.x, gMouseAtRotateStart.y); - } + this.xzDistanceToGrab = Vec3.length(xzOffset); } -function mousePressEvent(event) { +Grabber.prototype.pressEvent = function(event) { if (!event.isLeftButton) { return; } - gInitialMouse = {x: event.x, y: event.y }; - gPreviousMouse = {x: event.x, y: event.y }; var pickRay = Camera.computePickRay(event.x, event.y); var pickResults = Entities.findRayIntersection(pickRay, true); // accurate picking @@ -193,150 +217,172 @@ function mousePressEvent(event) { return; } + mouse.startDrag(event); + var clickedEntity = pickResults.entityID; var entityProperties = Entities.getEntityProperties(clickedEntity) - gStartPosition = entityProperties.position; - gStartRotation = entityProperties.rotation; + this.startPosition = entityProperties.position; + this.lastRotation = entityProperties.rotation; var cameraPosition = Camera.getPosition(); - gBeaconHeight = Vec3.length(entityProperties.dimensions); - gMaxGrabDistance = gBeaconHeight / MAX_SOLID_ANGLE; - if (Vec3.distance(gStartPosition, cameraPosition) > gMaxGrabDistance) { + var objectBoundingDiameter = Vec3.length(entityProperties.dimensions); + beacon.height = objectBoundingDiameter; + this.maxDistance = objectBoundingDiameter / MAX_SOLID_ANGLE; + if (Vec3.distance(this.startPosition, cameraPosition) > this.maxDistance) { // don't allow grabs of things far away return; } Entities.editEntity(clickedEntity, { gravity: ZERO_VEC3 }); - gIsGrabbing = true; + this.isGrabbing = true; - gGrabbedEntity = clickedEntity; - gCurrentPosition = entityProperties.position; - gOriginalGravity = entityProperties.gravity; - gTargetPosition = gStartPosition; + this.entityID = clickedEntity; + this.currentPosition = entityProperties.position; + this.originalGravity = entityProperties.gravity; + this.targetPosition = {x: this.startPosition.x, y: this.startPosition.y, z: this.startPosition.z}; // compute the grab point - var nearestPoint = Vec3.subtract(gStartPosition, cameraPosition); + var nearestPoint = Vec3.subtract(this.startPosition, cameraPosition); var distanceToGrab = Vec3.dot(nearestPoint, pickRay.direction); nearestPoint = Vec3.multiply(distanceToGrab, pickRay.direction); - gPointOnPlane = Vec3.sum(cameraPosition, nearestPoint); + this.pointOnPlane = Vec3.sum(cameraPosition, nearestPoint); // compute the grab offset (points from object center to point of grab) - gGrabOffset = Vec3.subtract(gPointOnPlane, gStartPosition); + this.offset = Vec3.subtract(this.pointOnPlane, this.startPosition); - computeNewGrabPlane(); + this.computeNewGrabPlane(); - updateDropLine(gStartPosition); + beacon.updatePosition(this.startPosition); // TODO: play sounds again when we aren't leaking AudioInjector threads //Audio.playSound(grabSound, { position: entityProperties.position, volume: VOLUME }); } -function mouseReleaseEvent() { - if (gIsGrabbing) { - if (Vec3.length(gOriginalGravity) != 0) { - Entities.editEntity(gGrabbedEntity, { gravity: gOriginalGravity }); +Grabber.prototype.releaseEvent = function() { + if (this.isGrabbing) { + if (Vec3.length(this.originalGravity) != 0) { + Entities.editEntity(this.entityID, { gravity: this.originalGravity}); } - gIsGrabbing = false - Entities.deleteAction(gGrabbedEntity, gActionID); - gActionID = null; + this.isGrabbing = false + Entities.deleteAction(this.entityID, this.actionID); + this.actionID = null; - Overlays.editOverlay(gBeacon, { visible: false }); + beacon.disable(); // TODO: play sounds again when we aren't leaking AudioInjector threads //Audio.playSound(releaseSound, { position: entityProperties.position, volume: VOLUME }); } } -function mouseMoveEvent(event) { - if (!gIsGrabbing) { +Grabber.prototype.moveEvent = function(event) { + if (!this.isGrabbing) { return; } + mouse.updateDrag(event); // see if something added/restored gravity - var entityProperties = Entities.getEntityProperties(gGrabbedEntity); + var entityProperties = Entities.getEntityProperties(this.entityID); if (Vec3.length(entityProperties.gravity) != 0) { - gOriginalGravity = entityProperties.gravity; + this.originalGravity = entityProperties.gravity; } - gCurrentPosition = entityProperties.position; + this.currentPosition = entityProperties.position; var actionArgs = {}; - if (gGrabMode === "rotate") { - var deltaMouse = { x: 0, y: 0 }; - var dx = event.x - gInitialMouse.x; - var dy = event.y - gInitialMouse.y; + if (this.mode === "rotate") { + var drag = mouse.getDrag(); var orientation = Camera.getOrientation(); - var dragOffset = Vec3.multiply(dx, Quat.getRight(orientation)); - dragOffset = Vec3.sum(dragOffset, Vec3.multiply(-dy, Quat.getUp(orientation))); + var dragOffset = Vec3.multiply(drag.x, Quat.getRight(orientation)); + dragOffset = Vec3.sum(dragOffset, Vec3.multiply(-drag.y, Quat.getUp(orientation))); var axis = Vec3.cross(dragOffset, Quat.getFront(orientation)); axis = Vec3.normalize(axis); var ROTATE_STRENGTH = 0.4; // magic number tuned by hand - var angle = ROTATE_STRENGTH * Math.sqrt((dx * dx) + (dy * dy)); + var angle = ROTATE_STRENGTH * Math.sqrt((drag.x * drag.x) + (drag.y * drag.y)); var deltaQ = Quat.angleAxis(angle, axis); // var qZero = entityProperties.rotation; - var qZero = gStartRotation; - var qOne = Quat.multiply(deltaQ, qZero); - actionArgs = {targetRotation: qOne, angularTimeScale: 0.1}; + //var qZero = this.lastRotation; + this.lastRotation = Quat.multiply(deltaQ, this.lastRotation); + actionArgs = {targetRotation: this.lastRotation, angularTimeScale: 0.1}; } else { - var newTargetPosition; - if (gGrabMode === "verticalCylinder") { + var newPointOnPlane; + if (this.mode === "verticalCylinder") { // for this mode we recompute the plane based on current Camera var planeNormal = Quat.getFront(Camera.getOrientation()); planeNormal.y = 0; planeNormal = Vec3.normalize(planeNormal); - var pointOnCylinder = Vec3.multiply(planeNormal, gXzDistanceToGrab); + var pointOnCylinder = Vec3.multiply(planeNormal, this.xzDistanceToGrab); pointOnCylinder = Vec3.sum(Camera.getPosition(), pointOnCylinder); - newTargetPosition = mouseIntersectionWithPlane(pointOnCylinder, planeNormal, event); - gPointOnPlane = Vec3.sum(newTargetPosition, gGrabOffset); + this.pointOnPlane = mouseIntersectionWithPlane(pointOnCylinder, planeNormal, mouse.current, this.maxDistance); + newPointOnPlane = {x: this.pointOnPlane.x, y: this.pointOnPlane.y, z: this.pointOnPlane.z}; } else { var cameraPosition = Camera.getPosition(); - newTargetPosition = mouseIntersectionWithPlane(gPointOnPlane, gPlaneNormal, event); - var relativePosition = Vec3.subtract(newTargetPosition, cameraPosition); + newPointOnPlane = mouseIntersectionWithPlane(this.pointOnPlane, this.planeNormal, mouse.current, this.maxDistance); + var relativePosition = Vec3.subtract(newPointOnPlane, cameraPosition); var distance = Vec3.length(relativePosition); - if (distance > gMaxGrabDistance) { + if (distance > this.maxDistance) { // clamp distance - relativePosition = Vec3.multiply(relativePosition, gMaxGrabDistance / distance); - newTargetPosition = Vec3.sum(relativePosition, cameraPosition); + relativePosition = Vec3.multiply(relativePosition, this.maxDistance / distance); + newPointOnPlane = Vec3.sum(relativePosition, cameraPosition); } } - gTargetPosition = Vec3.subtract(newTargetPosition, gGrabOffset); - actionArgs = {targetPosition: gTargetPosition, linearTimeScale: 0.1}; - } - gPreviousMouse = { x: event.x, y: event.y }; - gMouseCursorLocation = { x: Window.getCursorPositionX(), y: Window.getCursorPositionY() }; + this.targetPosition = Vec3.subtract(newPointOnPlane, this.offset); + actionArgs = {targetPosition: this.targetPosition, linearTimeScale: 0.1}; - if (!gActionID) { - gActionID = Entities.addAction("spring", gGrabbedEntity, actionArgs); + beacon.updatePosition(this.targetPosition); + } + + if (!this.actionID) { + this.actionID = Entities.addAction("spring", this.entityID, actionArgs); } else { - Entities.updateAction(gGrabbedEntity, gActionID, actionArgs); + Entities.updateAction(this.entityID, this.actionID, actionArgs); } - - updateDropLine(gTargetPosition); } -function keyReleaseEvent(event) { +Grabber.prototype.keyReleaseEvent = function(event) { if (event.text === "SHIFT") { - gLiftKey = false; + this.liftKey = false; } if (event.text === "CONTROL") { - gRotateKey = false; + this.rotateKey = false; } - computeNewGrabPlane(); + this.computeNewGrabPlane(); +} + +Grabber.prototype.keyPressEvent = function(event) { + if (event.text === "SHIFT") { + this.liftKey = true; + } + if (event.text === "CONTROL") { + this.rotateKey = true; + } + this.computeNewGrabPlane(); +} + +var grabber = new Grabber(); + +function pressEvent(event) { + grabber.pressEvent(event); +} + +function moveEvent(event) { + grabber.moveEvent(event); +} + +function releaseEvent(event) { + grabber.releaseEvent(event); } function keyPressEvent(event) { - if (event.text === "SHIFT") { - gLiftKey = true; - } - if (event.text === "CONTROL") { - gRotateKey = true; - } - computeNewGrabPlane(); + grabber.keyPressEvent(event); } -Controller.mouseMoveEvent.connect(mouseMoveEvent); -Controller.mousePressEvent.connect(mousePressEvent); -Controller.mouseReleaseEvent.connect(mouseReleaseEvent); +function keyReleaseEvent(event) { + grabber.keyReleaseEvent(event); +} + +Controller.mousePressEvent.connect(pressEvent); +Controller.mouseMoveEvent.connect(moveEvent); +Controller.mouseReleaseEvent.connect(releaseEvent); Controller.keyPressEvent.connect(keyPressEvent); Controller.keyReleaseEvent.connect(keyReleaseEvent); From f1c10f511915a0c45c52d703f2d27509f014fd67 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 13:38:12 -0700 Subject: [PATCH 10/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index 3cc26f72bc..7b67bb93b5 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -4,7 +4,7 @@ include(ExternalProject) ExternalProject_Add( ${EXTERNAL_NAME} URL http://hifi-public.s3.amazonaws.com/dependencies/polyvox-master-2015-7-15.zip - URL_MD5 9268bc5dc4141b303521dd67c133f1df + URL_MD5 9ec6323b87e849ae36e562ae1c7494a9 CMAKE_ARGS -DENABLE_EXAMPLES=OFF -DCMAKE_INSTALL_PREFIX:PATH= BINARY_DIR ${EXTERNAL_PROJECT_PREFIX}/build LOG_DOWNLOAD 1 From 930e570b48f55a04d7c450b6783d9250f50a1d8a Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 13:54:57 -0700 Subject: [PATCH 11/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index 7b67bb93b5..5840eaf624 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -52,11 +52,11 @@ if (WIN32) set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/PolyVoxCore/lib/Release/PolyVoxCore.lib CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/PolyVoxUtil/lib/PolyVoxUtil.lib CACHE FILEPATH "polyvox util library") elseif (APPLE) -set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") -set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/Debug/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/Release/libPolyVoxCore.dylib CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxUtil.dylib CACHE FILEPATH "polyvox util library") else () - set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") - set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_DEBUG ${INSTALL_DIR}/lib/Debug/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") + set(${EXTERNAL_NAME_UPPER}_CORE_LIBRARY_RELEASE ${INSTALL_DIR}/lib/Release/libPolyVoxCore.so CACHE FILEPATH "polyvox core library") # set(${EXTERNAL_NAME_UPPER}_UTIL_LIBRARY ${INSTALL_DIR}/lib/libPolyVoxUtil.so CACHE FILEPATH "polyvox util library") endif () From 4db66e7894532b242439ce46af950a41e46e2a35 Mon Sep 17 00:00:00 2001 From: Seth Alves Date: Wed, 15 Jul 2015 14:38:44 -0700 Subject: [PATCH 12/14] build separate debug and release versions of polyvox library --- cmake/externals/polyvox/CMakeLists.txt | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/cmake/externals/polyvox/CMakeLists.txt b/cmake/externals/polyvox/CMakeLists.txt index 5840eaf624..4563bf5918 100644 --- a/cmake/externals/polyvox/CMakeLists.txt +++ b/cmake/externals/polyvox/CMakeLists.txt @@ -24,7 +24,16 @@ if (APPLE) ${EXTERNAL_NAME} change-install-name COMMENT "Calling install_name_tool on libraries to fix install name for dylib linking" - COMMAND ${CMAKE_COMMAND} -DINSTALL_NAME_LIBRARY_DIR=${INSTALL_NAME_LIBRARY_DIR} -P ${EXTERNAL_PROJECT_DIR}/OSXInstallNameChange.cmake + COMMAND ${CMAKE_COMMAND} -DINSTALL_NAME_LIBRARY_DIR=${INSTALL_NAME_LIBRARY_DIR}/Debug -P ${EXTERNAL_PROJECT_DIR}/OSXInstallNameChange.cmake + DEPENDEES install + WORKING_DIRECTORY + LOG 1 + ) + ExternalProject_Add_Step( + ${EXTERNAL_NAME} + change-install-name + COMMENT "Calling install_name_tool on libraries to fix install name for dylib linking" + COMMAND ${CMAKE_COMMAND} -DINSTALL_NAME_LIBRARY_DIR=${INSTALL_NAME_LIBRARY_DIR}/Release -P ${EXTERNAL_PROJECT_DIR}/OSXInstallNameChange.cmake DEPENDEES install WORKING_DIRECTORY LOG 1 From 9657d146f7b04c9f7ccd35564d5ecd1455e0f836 Mon Sep 17 00:00:00 2001 From: Sam Gateau Date: Wed, 15 Jul 2015 16:09:53 -0700 Subject: [PATCH 13/14] Clearing Model.cpp from naked gl call references --- libraries/render-utils/src/Model.cpp | 32 ++++++---------------------- libraries/render-utils/src/Model.h | 1 - 2 files changed, 6 insertions(+), 27 deletions(-) diff --git a/libraries/render-utils/src/Model.cpp b/libraries/render-utils/src/Model.cpp index 6385041959..07df7fbda4 100644 --- a/libraries/render-utils/src/Model.cpp +++ b/libraries/render-utils/src/Model.cpp @@ -55,10 +55,6 @@ #include "model_lightmap_specular_map_frag.h" #include "model_translucent_frag.h" - -#define GLBATCH( call ) batch._##call -//#define GLBATCH( call ) call - using namespace std; static int modelPointerTypeId = qRegisterMetaType >(); @@ -1850,22 +1846,6 @@ void Model::deleteGeometry() { _blendedBlendshapeCoefficients.clear(); } -void Model::setupBatchTransform(gpu::Batch& batch, RenderArgs* args) { - - // Capture the view matrix once for the rendering of this model - if (_transforms.empty()) { - _transforms.push_back(Transform()); - } - - // We should be able to use the Frustum viewpoint onstead of the "viewTransform" - // but it s still buggy in some cases, so let's s wait and fix it... - _transforms[0] = _viewState->getViewTransform(); - - _transforms[0].preTranslate(-_translation); - - batch.setViewTransform(_transforms[0]); -} - AABox Model::getPartBounds(int meshIndex, int partIndex) { if (meshIndex < _meshStates.size()) { @@ -2000,7 +1980,7 @@ void Model::renderPart(RenderArgs* args, int meshIndex, int partIndex, bool tran } if (isSkinned) { - GLBATCH(glUniformMatrix4fv)(locations->clusterMatrices, state.clusterMatrices.size(), false, + batch._glUniformMatrix4fv(locations->clusterMatrices, state.clusterMatrices.size(), false, (const float*)state.clusterMatrices.constData()); _transforms[0] = Transform(); _transforms[0].preTranslate(_translation); @@ -2021,7 +2001,7 @@ void Model::renderPart(RenderArgs* args, int meshIndex, int partIndex, bool tran } if (mesh.colors.isEmpty()) { - GLBATCH(glColor4f)(1.0f, 1.0f, 1.0f, 1.0f); + batch._glColor4f(1.0f, 1.0f, 1.0f, 1.0f); } // guard against partially loaded meshes @@ -2077,7 +2057,7 @@ void Model::renderPart(RenderArgs* args, int meshIndex, int partIndex, bool tran if (!part.emissiveTexture.transform.isIdentity()) { part.emissiveTexture.transform.getMatrix(texcoordTransform[1]); } - GLBATCH(glUniformMatrix4fv)(locations->texcoordMatrices, 2, false, (const float*) &texcoordTransform); + batch._glUniformMatrix4fv(locations->texcoordMatrices, 2, false, (const float*) &texcoordTransform); } if (!mesh.tangents.isEmpty()) { @@ -2102,7 +2082,7 @@ void Model::renderPart(RenderArgs* args, int meshIndex, int partIndex, bool tran // assert(locations->emissiveParams >= 0); // we should have the emissiveParams defined in the shader float emissiveOffset = part.emissiveParams.x; float emissiveScale = part.emissiveParams.y; - GLBATCH(glUniform2f)(locations->emissiveParams, emissiveOffset, emissiveScale); + batch._glUniform2f(locations->emissiveParams, emissiveOffset, emissiveScale); NetworkTexture* emissiveMap = networkPart.emissiveTexture.data(); batch.setResourceTexture(locations->emissiveTextureUnit, (!emissiveMap || !emissiveMap->isLoaded()) ? @@ -2210,12 +2190,12 @@ void Model::pickPrograms(gpu::Batch& batch, RenderMode mode, bool translucent, f batch.setPipeline((*pipeline).second._pipeline); if ((locations->alphaThreshold > -1) && (mode != RenderArgs::SHADOW_RENDER_MODE)) { - GLBATCH(glUniform1f)(locations->alphaThreshold, alphaThreshold); + batch._glUniform1f(locations->alphaThreshold, alphaThreshold); } if ((locations->glowIntensity > -1) && (mode != RenderArgs::SHADOW_RENDER_MODE)) { const float DEFAULT_GLOW_INTENSITY = 1.0f; // FIXME - glow is removed - GLBATCH(glUniform1f)(locations->glowIntensity, DEFAULT_GLOW_INTENSITY); + batch._glUniform1f(locations->glowIntensity, DEFAULT_GLOW_INTENSITY); } } diff --git a/libraries/render-utils/src/Model.h b/libraries/render-utils/src/Model.h index b2d5ba7e4a..b3a62b8da7 100644 --- a/libraries/render-utils/src/Model.h +++ b/libraries/render-utils/src/Model.h @@ -407,7 +407,6 @@ private: // helper functions used by render() or renderInScene() - void setupBatchTransform(gpu::Batch& batch, RenderArgs* args); static void pickPrograms(gpu::Batch& batch, RenderArgs::RenderMode mode, bool translucent, float alphaThreshold, bool hasLightmap, bool hasTangents, bool hasSpecular, bool isSkinned, bool isWireframe, RenderArgs* args, Locations*& locations); From cf0940a65d992f4cb3d2bee703e2cacc8223ba11 Mon Sep 17 00:00:00 2001 From: Ryan Huffman Date: Thu, 16 Jul 2015 11:07:01 -0700 Subject: [PATCH 14/14] Fix edit.js highlighting an object after being turned off --- examples/edit.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/examples/edit.js b/examples/edit.js index 23cc942192..dfbace7be6 100644 --- a/examples/edit.js +++ b/examples/edit.js @@ -656,7 +656,9 @@ function mouseMove(event) { function handleIdleMouse() { idleMouseTimerId = null; - highlightEntityUnderCursor(lastMousePosition, true); + if (isActive) { + highlightEntityUnderCursor(lastMousePosition, true); + } } function highlightEntityUnderCursor(position, accurateRay) {