diff --git a/BUILD.md b/BUILD.md index e2f310bb1f..72dfa5725e 100644 --- a/BUILD.md +++ b/BUILD.md @@ -5,25 +5,30 @@ * [OpenSSL](https://www.openssl.org/related/binaries.html) ~> 1.0.1m * IMPORTANT: Using the recommended version of OpenSSL is critical to avoid security vulnerabilities. * [VHACD](https://github.com/virneo/v-hacd)(clone this repository)(Optional) -* [zlib](http://www.zlib.net/) ####CMake External Project Dependencies +* [boostconfig](https://github.com/boostorg/config) ~> 1.58 * [Bullet Physics Engine](https://code.google.com/p/bullet/downloads/list) ~> 2.82 -* [Intel Threading Building Blocks](https://www.threadingbuildingblocks.org/) ~> 4.3 +* [Faceshift](http://www.faceshift.com/) ~> 4.3 +* [GLEW](http://glew.sourceforge.net/) * [glm](http://glm.g-truc.net/0.9.5/index.html) ~> 0.9.5.4 * [gverb](https://github.com/highfidelity/gverb) - -The following external projects are optional dependencies. You can indicate to CMake that you would like to include them by passing -DGET_$NAME=1 when running a clean CMake build. For example, to get CMake to download and compile SDL2 you would pass -DGET_SDL2=1. - +* [Oculus SDK](https://developer.oculus.com/downloads/) ~> 0.6 (Win32) / 0.5 (Mac / Linux) +* [oglplus](http://oglplus.org/) ~> 0.63 +* [OpenVR](https://github.com/ValveSoftware/openvr) ~> 0.91 (Win32 only) +* [Polyvox](http://www.volumesoffun.com/) ~> 0.2.1 * [SDL2](https://www.libsdl.org/download-2.0.php) ~> 2.0.3 - * Enables game controller support in Interface +* [soxr](http://soxr.sourceforge.net) ~> 0.1.1 +* [Intel Threading Building Blocks](https://www.threadingbuildingblocks.org/) ~> 4.3 +* [Sixense](http://sixense.com/) ~> 071615 +* [zlib](http://www.zlib.net/) ~> 1.28 (Win32 only) The above dependencies will be downloaded, built, linked and included automatically by CMake where we require them. The CMakeLists files that handle grabbing each of the following external dependencies can be found in the [cmake/externals folder](cmake/externals). The resulting downloads, source files and binaries will be placed in the `build/ext` folder in each of the subfolders for each external project. These are not placed in your normal build tree when doing an out of source build so that they do not need to be re-downloaded and re-compiled every time the CMake build folder is cleared. Should you want to force a re-download and re-compile of a specific external, you can simply remove that directory from the appropriate subfolder in `build/ext`. Should you want to force a re-download and re-compile of all externals, just remove the `build/ext` folder. -If you would like to use a specific install of a dependency instead of the version that would be grabbed as a CMake ExternalProject, you can pass -DGET_$NAME=0 (where $NAME is the name of the subfolder in [cmake/externals](cmake/externals)) when you run CMake to tell it not to get that dependency as an external project. +If you would like to use a specific install of a dependency instead of the version that would be grabbed as a CMake ExternalProject, you can pass -DUSE_LOCAL_$NAME=0 (where $NAME is the name of the subfolder in [cmake/externals](cmake/externals)) when you run CMake to tell it not to get that dependency as an external project. ###OS Specific Build Guides * [BUILD_OSX.md](BUILD_OSX.md) - additional instructions for OS X. @@ -63,7 +68,7 @@ For example, to pass the QT_CMAKE_PREFIX_PATH variable during build file generat ####Finding Dependencies -The following applies for dependencies we do not grab via CMake ExternalProject (OpenSSL is an example), or for dependencies you have opted not to grab as a CMake ExternalProject (via -DGET_$NAME=0). The list of dependencies we grab by default as external projects can be found in [the CMake External Project Dependencies section](#cmake-external-project-dependencies). +The following applies for dependencies we do not grab via CMake ExternalProject (OpenSSL is an example), or for dependencies you have opted not to grab as a CMake ExternalProject (via -DUSE_LOCAL_$NAME=0). The list of dependencies we grab by default as external projects can be found in [the CMake External Project Dependencies section](#cmake-external-project-dependencies). You can point our [Cmake find modules](cmake/modules/) to the correct version of dependencies by setting one of the three following variables to the location of the correct version of the dependency. @@ -77,5 +82,5 @@ In the examples below the variable $NAME would be replaced by the name of the de ####Devices -You can support external input/output devices such as Oculus Rift, Leap Motion, Faceshift, MIDI, Razr Hydra and more by adding each individual SDK in the visible building path. Refer to the readme file available in each device folder in [interface/external/](interface/external) for the detailed explanation of the requirements to use the device. +You can support external input/output devices such as Leap Motion, MIDI, and more by adding each individual SDK in the visible building path. Refer to the readme file available in each device folder in [interface/external/](interface/external) for the detailed explanation of the requirements to use the device. diff --git a/BUILD_WIN.md b/BUILD_WIN.md index d291cef4f2..89646e99ff 100644 --- a/BUILD_WIN.md +++ b/BUILD_WIN.md @@ -75,16 +75,6 @@ To prevent these problems, install OpenSSL yourself. Download the following bina Install OpenSSL into the Windows system directory, to make sure that Qt uses the version that you've just installed, and not some other version. -####zlib - -Install zlib from - - [Zlib for Windows](http://gnuwin32.sourceforge.net/packages/zlib.htm) - -and fix a header file, as described here: - - [zlib zconf.h bug](http://sourceforge.net/p/gnuwin32/bugs/169/) - ###Build High Fidelity using Visual Studio Follow the same build steps from the CMake section of [BUILD.md](BUILD.md), but pass a different generator to CMake. diff --git a/CMakeLists.txt b/CMakeLists.txt index efe99b550b..d1c69e4f6b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -203,6 +203,7 @@ if (NOT ANDROID) add_subdirectory(interface) set_target_properties(interface PROPERTIES FOLDER "Apps") add_subdirectory(tests) + add_subdirectory(plugins) add_subdirectory(tools) endif () diff --git a/cmake/externals/glew/CMakeLists.txt b/cmake/externals/glew/CMakeLists.txt index 5f59cc2377..0a4a0abe71 100644 --- a/cmake/externals/glew/CMakeLists.txt +++ b/cmake/externals/glew/CMakeLists.txt @@ -7,14 +7,15 @@ endif () include(ExternalProject) ExternalProject_Add( ${EXTERNAL_NAME} - URL http://hifi-public.s3.amazonaws.com/dependencies/glew_simple.zip - URL_MD5 0507dc08337a82a5e7ecbc5417f92cc1 + URL http://hifi-public.s3.amazonaws.com/dependencies/glew_simple2.zip + URL_MD5 f05d858e8203c32b689da208ad8b39db CONFIGURE_COMMAND CMAKE_ARGS ${ANDROID_CMAKE_ARGS} -DCMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE} -DCMAKE_INSTALL_PREFIX:PATH= LOG_DOWNLOAD 1 LOG_CONFIGURE 1 LOG_BUILD 1 ) + # Hide this external target (for ide users) set_target_properties(${EXTERNAL_NAME} PROPERTIES FOLDER "hidden/externals") diff --git a/cmake/externals/zlib/CMakeLists.txt b/cmake/externals/zlib/CMakeLists.txt index 69dbca714d..0d279cc469 100644 --- a/cmake/externals/zlib/CMakeLists.txt +++ b/cmake/externals/zlib/CMakeLists.txt @@ -1,28 +1,31 @@ +set(EXTERNAL_NAME zlib) +string(TOUPPER ${EXTERNAL_NAME} EXTERNAL_NAME_UPPER) -if (WIN32) - set(EXTERNAL_NAME zlib) - string(TOUPPER ${EXTERNAL_NAME} EXTERNAL_NAME_UPPER) +include(ExternalProject) - include(ExternalProject) - ExternalProject_Add( - ${EXTERNAL_NAME} - URL http://zlib.net/zlib128.zip - URL_MD5 126f8676442ffbd97884eb4d6f32afb4 - INSTALL_COMMAND "" - LOG_DOWNLOAD 1 - ) +ExternalProject_Add( +${EXTERNAL_NAME} +URL http://zlib.net/zlib128.zip +CMAKE_ARGS -DCMAKE_INSTALL_PREFIX:PATH= +BINARY_DIR ${EXTERNAL_PROJECT_PREFIX}/build +LOG_DOWNLOAD 1 +LOG_CONFIGURE 1 +LOG_BUILD 1 +) + +# Hide this external target (for ide users) +set_target_properties(${EXTERNAL_NAME} PROPERTIES FOLDER "hidden/externals") - # Hide this external target (for ide users) - set_target_properties(${EXTERNAL_NAME} PROPERTIES FOLDER "hidden/externals") +ExternalProject_Get_Property(${EXTERNAL_NAME} INSTALL_DIR) +set(${EXTERNAL_NAME_UPPER}_INCLUDE_DIR ${INSTALL_DIR}/include CACHE PATH "List of zlib include directories") +set(${EXTERNAL_NAME_UPPER}_INCLUDE_DIRS ${${EXTERNAL_NAME_UPPER}_INCLUDE_DIR} CACHE PATH "List of zlib include directories") +set(${EXTERNAL_NAME_UPPER}_DLL_PATH ${INSTALL_DIR}/bin CACHE FILEPATH "Location of ZLib DLL") +set(${EXTERNAL_NAME_UPPER}_LIBRARY_RELEASE ${INSTALL_DIR}/lib/zlib.lib CACHE FILEPATH "Location of zlib release library") +set(${EXTERNAL_NAME_UPPER}_LIBRARY_DEBUG ${INSTALL_DIR}/lib/zlibd.lib CACHE FILEPATH "Location of zlib debug library") - ExternalProject_Get_Property(${EXTERNAL_NAME} SOURCE_DIR) +include(SelectLibraryConfigurations) +select_library_configurations(${EXTERNAL_NAME_UPPER}) - set(${EXTERNAL_NAME_UPPER}_INCLUDE_DIRS ${SOURCE_DIR}/include CACHE PATH "List of zlib include directories") - - ExternalProject_Get_Property(${EXTERNAL_NAME} BINARY_DIR) - - set(${EXTERNAL_NAME_UPPER}_DLL_PATH ${BINARY_DIR}/Release CACHE FILEPATH "Location of GLEW DLL") - set(${EXTERNAL_NAME_UPPER}_LIBRARY_RELEASE ${BINARY_DIR}/Release/zlib.lib CACHE FILEPATH "Location of ZLib release library") - set(${EXTERNAL_NAME_UPPER}_LIBRARY_DEBUG "" CACHE FILEPATH "Location of ZLib debug library") - -endif () \ No newline at end of file +# Force selected libraries into the cache +set(${EXTERNAL_NAME_UPPER}_LIBRARY ${${EXTERNAL_NAME_UPPER}_LIBRARY} CACHE FILEPATH "Location of zlib libraries") +set(${EXTERNAL_NAME_UPPER}_LIBRARIES ${${EXTERNAL_NAME_UPPER}_LIBRARIES} CACHE FILEPATH "Location of zlib libraries") diff --git a/cmake/macros/SetupHifiPlugin.cmake b/cmake/macros/SetupHifiPlugin.cmake new file mode 100644 index 0000000000..0ee94c7816 --- /dev/null +++ b/cmake/macros/SetupHifiPlugin.cmake @@ -0,0 +1,33 @@ +# +# Created by Bradley Austin Davis on 2015/10/25 +# Copyright 2015 High Fidelity, Inc. +# +# Distributed under the Apache License, Version 2.0. +# See the accompanying file LICENSE or http:#www.apache.org/licenses/LICENSE-2.0.html +# +macro(SETUP_HIFI_PLUGIN) + set(${TARGET_NAME}_SHARED 1) + setup_hifi_library(${ARGV}) + add_dependencies(interface ${TARGET_NAME}) + set_target_properties(${TARGET_NAME} PROPERTIES FOLDER "Plugins") + + if (APPLE) + set(PLUGIN_PATH "interface.app/Contents/MacOS/plugins") + else() + set(PLUGIN_PATH "plugins") + endif() + + # create the destination for the plugin binaries + add_custom_command( + TARGET ${TARGET_NAME} POST_BUILD + COMMAND "${CMAKE_COMMAND}" -E make_directory + "${CMAKE_BINARY_DIR}/interface/$/${PLUGIN_PATH}/" + ) + + add_custom_command(TARGET ${DIR} POST_BUILD + COMMAND "${CMAKE_COMMAND}" -E copy + "$" + "${CMAKE_BINARY_DIR}/interface/$/${PLUGIN_PATH}/" + ) + +endmacro() \ No newline at end of file diff --git a/cmake/macros/TargetZlib.cmake b/cmake/macros/TargetZlib.cmake new file mode 100644 index 0000000000..3eb0322ea0 --- /dev/null +++ b/cmake/macros/TargetZlib.cmake @@ -0,0 +1,22 @@ +# +# Copyright 2015 High Fidelity, Inc. +# Created by Bradley Austin Davis on 2015/10/10 +# +# Distributed under the Apache License, Version 2.0. +# See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +# +macro(TARGET_ZLIB) + + if (WIN32) + add_dependency_external_projects(zlib) + endif() + + find_package(ZLIB REQUIRED) + + if (WIN32) + add_paths_to_fixup_libs(${ZLIB_DLL_PATH}) + endif() + + target_include_directories(${TARGET_NAME} SYSTEM PRIVATE ${ZLIB_INCLUDE_DIRS}) + target_link_libraries(${TARGET_NAME} ${ZLIB_LIBRARIES}) +endmacro() diff --git a/cmake/modules/FindGLEW.cmake b/cmake/modules/FindGLEW.cmake index 4f22271e3e..f4eca0eddf 100644 --- a/cmake/modules/FindGLEW.cmake +++ b/cmake/modules/FindGLEW.cmake @@ -30,6 +30,4 @@ include(SelectLibraryConfigurations) select_library_configurations(GLEW) include(FindPackageHandleStandardArgs) -find_package_handle_standard_args(GLEW DEFAULT_MSG GLEW_INCLUDE_DIRS GLEW_LIBRARIES) - -message(STATUS "Found GLEW - Assuming that GLEW is static and defining GLEW_STATIC") \ No newline at end of file +find_package_handle_standard_args(GLEW DEFAULT_MSG GLEW_INCLUDE_DIRS GLEW_LIBRARIES) \ No newline at end of file diff --git a/cmake/templates/FixupBundlePostBuild.cmake.in b/cmake/templates/FixupBundlePostBuild.cmake.in index 7d98e9796c..4afe4de403 100644 --- a/cmake/templates/FixupBundlePostBuild.cmake.in +++ b/cmake/templates/FixupBundlePostBuild.cmake.in @@ -10,5 +10,35 @@ # include(BundleUtilities) + + +# replace copy_resolved_item_into_bundle +# +# The official version of copy_resolved_item_into_bundle will print out a "warning:" when +# the resolved item matches the resolved embedded item. This not not really an issue that +# should rise to the level of a "warning" so we replace this message with a "status:" +# +function(copy_resolved_item_into_bundle resolved_item resolved_embedded_item) + if(WIN32) + # ignore case on Windows + string(TOLOWER "${resolved_item}" resolved_item_compare) + string(TOLOWER "${resolved_embedded_item}" resolved_embedded_item_compare) + else() + set(resolved_item_compare "${resolved_item}") + set(resolved_embedded_item_compare "${resolved_embedded_item}") + endif() + + if("${resolved_item_compare}" STREQUAL "${resolved_embedded_item_compare}") + # this is our only change from the original version + message(STATUS "status: resolved_item == resolved_embedded_item - not copying...") + else() + #message(STATUS "copying COMMAND ${CMAKE_COMMAND} -E copy ${resolved_item} ${resolved_embedded_item}") + execute_process(COMMAND ${CMAKE_COMMAND} -E copy "${resolved_item}" "${resolved_embedded_item}") + if(UNIX AND NOT APPLE) + file(RPATH_REMOVE FILE "${resolved_embedded_item}") + endif() + endif() +endfunction() + message(STATUS "FIXUP_LIBS for fixup_bundle called for bundle ${BUNDLE_EXECUTABLE} are @FIXUP_LIBS@") fixup_bundle("${BUNDLE_EXECUTABLE}" "" "@FIXUP_LIBS@") \ No newline at end of file diff --git a/examples/controllers/handControllerGrab.js b/examples/controllers/handControllerGrab.js index 339c1ad002..9f7d579f10 100644 --- a/examples/controllers/handControllerGrab.js +++ b/examples/controllers/handControllerGrab.js @@ -910,8 +910,8 @@ var leftController = new MyController(LEFT_HAND); var MAPPING_NAME = "com.highfidelity.handControllerGrab"; var mapping = Controller.newMapping(MAPPING_NAME); -mapping.from([Controller.Standard.RB, Controller.Standard.RT]).to(rightController.eitherTrigger); -mapping.from([Controller.Standard.LB, Controller.Standard.LT]).to(leftController.eitherTrigger); +mapping.from([Controller.Standard.RB, Controller.Standard.RT]).peek().to(rightController.eitherTrigger); +mapping.from([Controller.Standard.LB, Controller.Standard.LT]).peek().to(leftController.eitherTrigger); Controller.enableMapping(MAPPING_NAME); diff --git a/interface/resources/controllers/keyboardMouse.json b/interface/resources/controllers/keyboardMouse.json index 8af6b1dc98..9c3ba79d76 100644 --- a/interface/resources/controllers/keyboardMouse.json +++ b/interface/resources/controllers/keyboardMouse.json @@ -81,6 +81,7 @@ { "from": "Keyboard.Space", "to": "Actions.SHIFT" }, { "from": "Keyboard.R", "to": "Actions.ACTION1" }, - { "from": "Keyboard.T", "to": "Actions.ACTION2" } + { "from": "Keyboard.T", "to": "Actions.ACTION2" }, + { "from": "Keyboard.RightMouseClick", "to": "Actions.ContextMenu" } ] } diff --git a/interface/resources/controllers/standard.json b/interface/resources/controllers/standard.json index 871374b85b..b08abcbaad 100644 --- a/interface/resources/controllers/standard.json +++ b/interface/resources/controllers/standard.json @@ -16,7 +16,7 @@ { "from": "Standard.RX", "to": "Actions.Yaw" }, - { "from": "Standard.RY", "to": "Actions.Pitch" }, + { "from": "Standard.RY", "filters": "invert", "to": "Actions.TranslateY" }, { "from": [ "Standard.DU", "Standard.DL", "Standard.DR", "Standard.DD" ], "to": "Standard.LeftPrimaryThumb" }, @@ -25,6 +25,9 @@ { "from": [ "Standard.A", "Standard.B", "Standard.X", "Standard.Y" ], "to": "Standard.RightPrimaryThumb" }, { "from": "Standard.Start", "to": "Standard.RightSecondaryThumb" }, + { "from": "Standard.LeftSecondaryThumb", "to": "Actions.CycleCamera" }, + { "from": "Standard.RightSecondaryThumb", "to": "Actions.ContextMenu" }, + { "from": "Standard.LT", "to": "Actions.LeftHandClick" }, { "from": "Standard.RT", "to": "Actions.RightHandClick" }, diff --git a/interface/src/Application.cpp b/interface/src/Application.cpp index 7f4b5a3c3d..6d41679bea 100644 --- a/interface/src/Application.cpp +++ b/interface/src/Application.cpp @@ -392,7 +392,6 @@ Application::Application(int& argc, char** argv, QElapsedTimer& startupTimer) : _entityClipboard->createRootElement(); _pluginContainer = new PluginContainerProxy(); - Plugin::setContainer(_pluginContainer); #ifdef Q_OS_WIN installNativeEventFilter(&MyNativeEventFilter::getInstance()); #endif @@ -630,8 +629,14 @@ Application::Application(int& argc, char** argv, QElapsedTimer& startupTimer) : // Setup the userInputMapper with the actions auto userInputMapper = DependencyManager::get(); connect(userInputMapper.data(), &UserInputMapper::actionEvent, [this](int action, float state) { - if (state && action == toInt(controller::Action::TOGGLE_MUTE)) { - DependencyManager::get()->toggleMute(); + if (state) { + if (action == controller::toInt(controller::Action::TOGGLE_MUTE)) { + DependencyManager::get()->toggleMute(); + } else if (action == controller::toInt(controller::Action::CYCLE_CAMERA)) { + cycleCamera(); + } else if (action == controller::toInt(controller::Action::CONTEXT_MENU)) { + VrMenu::toggle(); // show context menu even on non-stereo displays + } } }); @@ -1785,9 +1790,7 @@ void Application::keyPressEvent(QKeyEvent* event) { void Application::keyReleaseEvent(QKeyEvent* event) { if (event->key() == Qt::Key_Alt && _altPressed && hasFocus()) { - if (getActiveDisplayPlugin()->isStereo()) { - VrMenu::toggle(); - } + VrMenu::toggle(); // show context menu even on non-stereo displays } _keysPressed.remove(event->key()); @@ -2620,6 +2623,30 @@ void Application::updateThreads(float deltaTime) { } } +void Application::cycleCamera() { + auto menu = Menu::getInstance(); + if (menu->isOptionChecked(MenuOption::FullscreenMirror)) { + + menu->setIsOptionChecked(MenuOption::FullscreenMirror, false); + menu->setIsOptionChecked(MenuOption::FirstPerson, true); + + } else if (menu->isOptionChecked(MenuOption::FirstPerson)) { + + menu->setIsOptionChecked(MenuOption::FirstPerson, false); + menu->setIsOptionChecked(MenuOption::ThirdPerson, true); + + } else if (menu->isOptionChecked(MenuOption::ThirdPerson)) { + + menu->setIsOptionChecked(MenuOption::ThirdPerson, false); + menu->setIsOptionChecked(MenuOption::FullscreenMirror, true); + + } else if (menu->isOptionChecked(MenuOption::IndependentMode)) { + // do nothing if in independe mode + return; + } + cameraMenuChanged(); // handle the menu change +} + void Application::cameraMenuChanged() { if (Menu::getInstance()->isOptionChecked(MenuOption::FullscreenMirror)) { if (_myCamera.getMode() != CAMERA_MODE_MIRROR) { diff --git a/interface/src/Application.h b/interface/src/Application.h index bf45a0c6ec..dfc904e0ef 100644 --- a/interface/src/Application.h +++ b/interface/src/Application.h @@ -293,8 +293,9 @@ public slots: void aboutApp(); void showEditEntitiesHelp(); + void cycleCamera(); void cameraMenuChanged(); - + void reloadResourceCaches(); void crashApplication(); diff --git a/interface/src/PluginContainerProxy.cpp b/interface/src/PluginContainerProxy.cpp index 079d6d0bea..2e5c883897 100644 --- a/interface/src/PluginContainerProxy.cpp +++ b/interface/src/PluginContainerProxy.cpp @@ -13,7 +13,6 @@ #include "ui/DialogsManager.h" PluginContainerProxy::PluginContainerProxy() { - Plugin::setContainer(this); } PluginContainerProxy::~PluginContainerProxy() { diff --git a/libraries/controllers/src/controllers/Actions.cpp b/libraries/controllers/src/controllers/Actions.cpp index a4a8656fb7..701aea79c5 100644 --- a/libraries/controllers/src/controllers/Actions.cpp +++ b/libraries/controllers/src/controllers/Actions.cpp @@ -60,6 +60,7 @@ namespace controller { makeButtonPair(Action::ACTION2, "SecondaryAction"), makeButtonPair(Action::CONTEXT_MENU, "ContextMenu"), makeButtonPair(Action::TOGGLE_MUTE, "ToggleMute"), + makeButtonPair(Action::CYCLE_CAMERA, "CycleCamera"), // Aliases and bisected versions makeAxisPair(Action::LONGITUDINAL_BACKWARD, "Backward"), diff --git a/libraries/controllers/src/controllers/Actions.h b/libraries/controllers/src/controllers/Actions.h index 36df695032..d5ad44c720 100644 --- a/libraries/controllers/src/controllers/Actions.h +++ b/libraries/controllers/src/controllers/Actions.h @@ -51,6 +51,7 @@ enum class Action { CONTEXT_MENU, TOGGLE_MUTE, + CYCLE_CAMERA, SHIFT, diff --git a/libraries/controllers/src/controllers/DeviceProxy.cpp b/libraries/controllers/src/controllers/DeviceProxy.cpp index f3e9526080..f03354c52d 100644 --- a/libraries/controllers/src/controllers/DeviceProxy.cpp +++ b/libraries/controllers/src/controllers/DeviceProxy.cpp @@ -7,8 +7,11 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "DeviceProxy.h" - -namespace controller { -} +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "DeviceProxy.h" diff --git a/libraries/controllers/src/controllers/StandardController.cpp b/libraries/controllers/src/controllers/StandardController.cpp index 44f1bff1ae..31c023b1ea 100644 --- a/libraries/controllers/src/controllers/StandardController.cpp +++ b/libraries/controllers/src/controllers/StandardController.cpp @@ -118,41 +118,5 @@ QString StandardController::getDefaultMappingConfig() const { return DEFAULT_MAPPING_JSON; } -// FIXME figure out how to move the shifted version to JSON -//void StandardController::assignDefaultInputMapping(UserInputMapper& mapper) { -// const float JOYSTICK_MOVE_SPEED = 1.0f; -// const float DPAD_MOVE_SPEED = 0.5f; -// const float JOYSTICK_YAW_SPEED = 0.5f; -// const float JOYSTICK_PITCH_SPEED = 0.25f; -// const float BOOM_SPEED = 0.1f; -// -// // Hold front right shoulder button for precision controls -// // Left Joystick: Movement, strafing -// mapper.addInputChannel(UserInputMapper::TRANSLATE_Z, makeInput(controller::LY), makeInput(controller::RB), JOYSTICK_MOVE_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::TRANSLATE_X, makeInput(controller::LY), makeInput(controller::RB), JOYSTICK_MOVE_SPEED / 2.0f); -// -// // Right Joystick: Camera orientation -// mapper.addInputChannel(UserInputMapper::YAW, makeInput(controller::RX), makeInput(controller::RB), JOYSTICK_YAW_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::PITCH, makeInput(controller::RY), makeInput(controller::RB), JOYSTICK_PITCH_SPEED / 2.0f); -// -// // Dpad movement -// mapper.addInputChannel(UserInputMapper::LONGITUDINAL_FORWARD, makeInput(controller::DU), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::LONGITUDINAL_BACKWARD, makeInput(controller::DD), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::LATERAL_RIGHT, makeInput(controller::DR), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::LATERAL_LEFT, makeInput(controller::DL), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// -// // Button controls -// mapper.addInputChannel(UserInputMapper::VERTICAL_UP, makeInput(controller::Y), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::VERTICAL_DOWN, makeInput(controller::X), makeInput(controller::RB), DPAD_MOVE_SPEED / 2.0f); -// -// // Zoom -// mapper.addInputChannel(UserInputMapper::BOOM_IN, makeInput(controller::RT), makeInput(controller::RB), BOOM_SPEED / 2.0f); -// mapper.addInputChannel(UserInputMapper::BOOM_OUT, makeInput(controller::LT), makeInput(controller::RB), BOOM_SPEED / 2.0f); -// -// mapper.addInputChannel(UserInputMapper::SHIFT, makeInput(controller::RB)); -// -// mapper.addInputChannel(UserInputMapper::ACTION1, makeInput(controller::B)); -// mapper.addInputChannel(UserInputMapper::ACTION2, makeInput(controller::A)); -//} } diff --git a/libraries/controllers/src/controllers/UserInputMapper.cpp b/libraries/controllers/src/controllers/UserInputMapper.cpp index 8e121de7fb..1a0fae8158 100755 --- a/libraries/controllers/src/controllers/UserInputMapper.cpp +++ b/libraries/controllers/src/controllers/UserInputMapper.cpp @@ -513,7 +513,7 @@ bool UserInputMapper::applyRoute(const Route::Pointer& route, bool force) { // and someone else wires it to CONTEXT_MENU, I don't want both to occur when // I press the button. The exception is if I'm wiring a control back to itself // in order to adjust my interface, like inverting the Y axis on an analog stick - if (!source->readable()) { + if (!route->peek && !source->readable()) { if (debugRoutes && route->debug) { qCDebug(controllers) << "Source unreadable"; } @@ -539,7 +539,7 @@ bool UserInputMapper::applyRoute(const Route::Pointer& route, bool force) { // Fetch the value, may have been overriden by previous loopback routes if (source->isPose()) { - Pose value = getPose(source); + Pose value = getPose(source, route->peek); static const Pose IDENTITY_POSE { vec3(), quat() }; if (debugRoutes && route->debug) { if (!value.valid) { @@ -554,7 +554,7 @@ bool UserInputMapper::applyRoute(const Route::Pointer& route, bool force) { destination->apply(value, source); } else { // Fetch the value, may have been overriden by previous loopback routes - float value = getValue(source); + float value = getValue(source, route->peek); if (debugRoutes && route->debug) { qCDebug(controllers) << "Value was " << value; @@ -691,8 +691,8 @@ void UserInputMapper::enableMapping(const QString& mappingName, bool enable) { } } -float UserInputMapper::getValue(const Endpoint::Pointer& endpoint) { - return endpoint->value(); +float UserInputMapper::getValue(const Endpoint::Pointer& endpoint, bool peek) { + return peek ? endpoint->peek() : endpoint->value(); } float UserInputMapper::getValue(const Input& input) const { @@ -704,11 +704,11 @@ float UserInputMapper::getValue(const Input& input) const { return endpoint->value(); } -Pose UserInputMapper::getPose(const Endpoint::Pointer& endpoint) { +Pose UserInputMapper::getPose(const Endpoint::Pointer& endpoint, bool peek) { if (!endpoint->isPose()) { return Pose(); } - return endpoint->pose(); + return peek ? endpoint->peekPose() : endpoint->pose(); } Pose UserInputMapper::getPose(const Input& input) const { @@ -742,6 +742,7 @@ static const QString JSON_NAME = QStringLiteral("name"); static const QString JSON_CHANNELS = QStringLiteral("channels"); static const QString JSON_CHANNEL_FROM = QStringLiteral("from"); static const QString JSON_CHANNEL_DEBUG = QStringLiteral("debug"); +static const QString JSON_CHANNEL_PEEK = QStringLiteral("peek"); static const QString JSON_CHANNEL_WHEN = QStringLiteral("when"); static const QString JSON_CHANNEL_TO = QStringLiteral("to"); static const QString JSON_CHANNEL_FILTERS = QStringLiteral("filters"); @@ -953,6 +954,7 @@ Route::Pointer UserInputMapper::parseRoute(const QJsonValue& value) { result->json = QString(QJsonDocument(obj).toJson()); result->source = parseSource(obj[JSON_CHANNEL_FROM]); result->debug = obj[JSON_CHANNEL_DEBUG].toBool(); + result->debug = obj[JSON_CHANNEL_PEEK].toBool(); if (!result->source) { qWarning() << "Invalid route source " << obj[JSON_CHANNEL_FROM]; return Route::Pointer(); diff --git a/libraries/controllers/src/controllers/UserInputMapper.h b/libraries/controllers/src/controllers/UserInputMapper.h index 7684ecb7c5..c1dfcf5d33 100644 --- a/libraries/controllers/src/controllers/UserInputMapper.h +++ b/libraries/controllers/src/controllers/UserInputMapper.h @@ -38,8 +38,8 @@ namespace controller { class UserInputMapper : public QObject, public Dependency { Q_OBJECT - SINGLETON_DEPENDENCY - Q_ENUMS(Action) + SINGLETON_DEPENDENCY + Q_ENUMS(Action) public: // FIXME move to unordered set / map @@ -135,8 +135,8 @@ namespace controller { int recordDeviceOfType(const QString& deviceName); QHash _deviceCounts; - static float getValue(const EndpointPointer& endpoint); - static Pose getPose(const EndpointPointer& endpoint); + static float getValue(const EndpointPointer& endpoint, bool peek = false); + static Pose getPose(const EndpointPointer& endpoint, bool peek = false); friend class RouteBuilderProxy; friend class MappingBuilderProxy; diff --git a/libraries/controllers/src/controllers/impl/Endpoint.cpp b/libraries/controllers/src/controllers/impl/Endpoint.cpp index 9e9b13f8ea..e771b1916f 100644 --- a/libraries/controllers/src/controllers/impl/Endpoint.cpp +++ b/libraries/controllers/src/controllers/impl/Endpoint.cpp @@ -6,9 +6,11 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "Endpoint.h" - -namespace controller { - -} +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "Endpoint.h" diff --git a/libraries/controllers/src/controllers/impl/Endpoint.h b/libraries/controllers/src/controllers/impl/Endpoint.h index bc604da2c5..475dc035bb 100644 --- a/libraries/controllers/src/controllers/impl/Endpoint.h +++ b/libraries/controllers/src/controllers/impl/Endpoint.h @@ -36,12 +36,13 @@ namespace controller { using WriteLambda = std::function; Endpoint(const Input& input) : _input(input) {} - virtual float value() = 0; + virtual float value() { return peek(); } + virtual float peek() const = 0; virtual void apply(float value, const Pointer& source) = 0; - virtual Pose pose() { return Pose(); } + virtual Pose peekPose() const { return Pose(); }; + virtual Pose pose() { return peekPose(); } virtual void apply(const Pose& value, const Pointer& source) {} - virtual bool isPose() { return _input.isPose(); } - + virtual bool isPose() const { return _input.isPose(); } virtual bool writeable() const { return true; } virtual bool readable() const { return true; } virtual void reset() { } @@ -58,7 +59,7 @@ namespace controller { LambdaEndpoint(ReadLambda readLambda, WriteLambda writeLambda = [](float) {}) : Endpoint(Input::INVALID_INPUT), _readLambda(readLambda), _writeLambda(writeLambda) { } - virtual float value() override { return _readLambda(); } + virtual float peek() const override { return _readLambda(); } virtual void apply(float value, const Pointer& source) override { _writeLambda(value); } private: @@ -73,10 +74,10 @@ namespace controller { : Endpoint(id) { } - virtual float value() override { return _currentValue; } + virtual float peek() const override { return _currentValue; } virtual void apply(float value, const Pointer& source) override { _currentValue = value; } - virtual Pose pose() override { return _currentPose; } + virtual Pose peekPose() const override { return _currentPose; } virtual void apply(const Pose& value, const Pointer& source) override { _currentPose = value; } diff --git a/libraries/controllers/src/controllers/impl/Mapping.cpp b/libraries/controllers/src/controllers/impl/Mapping.cpp index 68c43da393..dd8e1c1d48 100644 --- a/libraries/controllers/src/controllers/impl/Mapping.cpp +++ b/libraries/controllers/src/controllers/impl/Mapping.cpp @@ -5,4 +5,11 @@ // Distributed under the Apache License, Version 2.0. // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "Mapping.h" + +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "Mapping.h" diff --git a/libraries/controllers/src/controllers/impl/Route.cpp b/libraries/controllers/src/controllers/impl/Route.cpp index 56590e564a..c74f809195 100644 --- a/libraries/controllers/src/controllers/impl/Route.cpp +++ b/libraries/controllers/src/controllers/impl/Route.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "Route.h" +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "Route.h" diff --git a/libraries/controllers/src/controllers/impl/Route.h b/libraries/controllers/src/controllers/impl/Route.h index 5ad3d36628..554dd82d5a 100644 --- a/libraries/controllers/src/controllers/impl/Route.h +++ b/libraries/controllers/src/controllers/impl/Route.h @@ -26,6 +26,7 @@ namespace controller { Filter::List filters; QString json; bool debug { false }; + bool peek { false }; using Pointer = std::shared_ptr; using List = std::list; diff --git a/libraries/controllers/src/controllers/impl/RouteBuilderProxy.cpp b/libraries/controllers/src/controllers/impl/RouteBuilderProxy.cpp index 49e615439d..5ae52893e0 100644 --- a/libraries/controllers/src/controllers/impl/RouteBuilderProxy.cpp +++ b/libraries/controllers/src/controllers/impl/RouteBuilderProxy.cpp @@ -52,6 +52,11 @@ QObject* RouteBuilderProxy::debug(bool enable) { return this; } +QObject* RouteBuilderProxy::peek(bool enable) { + _route->peek = enable; + return this; +} + QObject* RouteBuilderProxy::when(const QScriptValue& expression) { _route->conditional = _parent.conditionalFor(expression); return this; diff --git a/libraries/controllers/src/controllers/impl/RouteBuilderProxy.h b/libraries/controllers/src/controllers/impl/RouteBuilderProxy.h index d55aa80f6b..1c0ed6931d 100644 --- a/libraries/controllers/src/controllers/impl/RouteBuilderProxy.h +++ b/libraries/controllers/src/controllers/impl/RouteBuilderProxy.h @@ -38,6 +38,7 @@ class RouteBuilderProxy : public QObject { Q_INVOKABLE void to(const QScriptValue& destination); Q_INVOKABLE QObject* debug(bool enable = true); + Q_INVOKABLE QObject* peek(bool enable = true); Q_INVOKABLE QObject* when(const QScriptValue& expression); Q_INVOKABLE QObject* clamp(float min, float max); Q_INVOKABLE QObject* hysteresis(float min, float max); @@ -48,7 +49,7 @@ class RouteBuilderProxy : public QObject { Q_INVOKABLE QObject* constrainToInteger(); Q_INVOKABLE QObject* constrainToPositiveInteger(); -private: + private: void to(const Endpoint::Pointer& destination); void conditional(const Conditional::Pointer& conditional); void addFilter(Filter::Pointer filter); diff --git a/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.cpp b/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.cpp index 03e16b8cf9..f833eedb60 100644 --- a/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.cpp +++ b/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "EndpointConditional.h" \ No newline at end of file +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "EndpointConditional.h" \ No newline at end of file diff --git a/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.h b/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.h index 54ed57e871..0ba1347087 100644 --- a/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.h +++ b/libraries/controllers/src/controllers/impl/conditionals/EndpointConditional.h @@ -18,7 +18,7 @@ namespace controller { class EndpointConditional : public Conditional { public: EndpointConditional(Endpoint::Pointer endpoint) : _endpoint(endpoint) {} - virtual bool satisfied() override { return _endpoint && _endpoint->value() != 0.0f; } + virtual bool satisfied() override { return _endpoint && _endpoint->peek() != 0.0f; } private: Endpoint::Pointer _endpoint; }; diff --git a/libraries/controllers/src/controllers/impl/conditionals/NotConditional.cpp b/libraries/controllers/src/controllers/impl/conditionals/NotConditional.cpp index 0c8d602b9e..e30b060985 100644 --- a/libraries/controllers/src/controllers/impl/conditionals/NotConditional.cpp +++ b/libraries/controllers/src/controllers/impl/conditionals/NotConditional.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "NotConditional.h" +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "NotConditional.h" diff --git a/libraries/controllers/src/controllers/impl/endpoints/ActionEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/ActionEndpoint.h index 574fdcedb5..e07dc9e4c8 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/ActionEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/ActionEndpoint.h @@ -23,10 +23,10 @@ class ActionEndpoint : public Endpoint { public: ActionEndpoint(const Input& id = Input::INVALID_INPUT) : Endpoint(id) { } - virtual float value() override { return _currentValue; } + virtual float peek() const { return _currentValue; } virtual void apply(float newValue, const Pointer& source) override; - virtual Pose pose() override { return _currentPose; } + virtual Pose peekPose() const { return _currentPose; } virtual void apply(const Pose& value, const Pointer& source) override; virtual void reset() override; diff --git a/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.cpp index 24f3479ea8..a3b719b2c1 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.cpp @@ -27,13 +27,19 @@ AnyEndpoint::AnyEndpoint(Endpoint::List children) : Endpoint(Input::INVALID_INPU } } +float AnyEndpoint::peek() const { + float result = 0; + for (auto& child : _children) { + result = std::max(result, child->peek()); + } + return result; +} + +// Fetching the value must trigger any necessary side effects of value() on ALL the children. float AnyEndpoint::value() { float result = 0; for (auto& child : _children) { - float childResult = child->value(); - if (childResult != 0.0f) { - result = childResult; - } + result = std::max(result, child->value()); } return result; } diff --git a/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.h index 24834ce223..8947eb675f 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/AnyEndpoint.h @@ -19,6 +19,7 @@ class AnyEndpoint : public Endpoint { public: using Endpoint::apply; AnyEndpoint(Endpoint::List children); + virtual float peek() const override; virtual float value() override; virtual void apply(float newValue, const Endpoint::Pointer& source) override; virtual bool writeable() const override; diff --git a/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.cpp index c083a7147d..5dea1de34e 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "ArrayEndpoint.h" \ No newline at end of file +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "ArrayEndpoint.h" \ No newline at end of file diff --git a/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.h index 899fa46de0..34d30a2e97 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/ArrayEndpoint.h @@ -21,9 +21,7 @@ public: using Pointer = std::shared_ptr; ArrayEndpoint() : Endpoint(Input::INVALID_INPUT) { } - virtual float value() override { - return 0.0; - } + virtual float peek() const override { return 0.0f; } virtual void apply(float value, const Endpoint::Pointer& source) override { for (auto& child : _children) { diff --git a/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.cpp index 913bf0136b..1bd27489f8 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.cpp @@ -24,6 +24,12 @@ bool CompositeEndpoint::readable() const { return first->readable() && second->readable(); } +float CompositeEndpoint::peek() const { + float result = first->peek() * -1.0f + second->peek(); + return result; +} + +// Fetching via value() must trigger any side effects of value() on the children float CompositeEndpoint::value() { float result = first->value() * -1.0f + second->value(); return result; diff --git a/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.h index b29266464c..3249aa1d37 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/CompositeEndpoint.h @@ -18,6 +18,7 @@ namespace controller { using Endpoint::apply; CompositeEndpoint(Endpoint::Pointer first, Endpoint::Pointer second); + virtual float peek() const override; virtual float value() override; virtual void apply(float newValue, const Pointer& source) override; virtual bool readable() const override; diff --git a/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.cpp index bb1f6df191..ce58c948d1 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.cpp @@ -13,10 +13,10 @@ #include "../../UserInputMapper.h" using namespace controller; -float InputEndpoint::value(){ - _read = true; + +float InputEndpoint::peek() const { if (isPose()) { - return pose().valid ? 1.0f : 0.0f; + return peekPose().valid ? 1.0f : 0.0f; } auto userInputMapper = DependencyManager::get(); auto deviceProxy = userInputMapper->getDevice(_input); @@ -26,8 +26,12 @@ float InputEndpoint::value(){ return deviceProxy->getValue(_input); } -Pose InputEndpoint::pose() { +float InputEndpoint::value(){ _read = true; + return peek(); +} + +Pose InputEndpoint::peekPose() const { if (!isPose()) { return Pose(); } @@ -39,3 +43,8 @@ Pose InputEndpoint::pose() { return deviceProxy->getPose(_input.channel); } +Pose InputEndpoint::pose() { + _read = true; + return peekPose(); +} + diff --git a/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.h index d58f0c2e73..663168bedc 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/InputEndpoint.h @@ -20,9 +20,11 @@ public: : Endpoint(id) { } + virtual float peek() const override; virtual float value() override; // FIXME need support for writing back to vibration / force feedback effects virtual void apply(float newValue, const Pointer& source) override {} + virtual Pose peekPose() const override; virtual Pose pose() override; virtual void apply(const Pose& value, const Pointer& source) override { } diff --git a/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.cpp index 4560741d12..7f0e80cbae 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "JSEndpoint.h" \ No newline at end of file +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "JSEndpoint.h" \ No newline at end of file diff --git a/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.h index 5113fef657..958914264e 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/JSEndpoint.h @@ -24,9 +24,8 @@ public: : Endpoint(Input::INVALID_INPUT), _callable(callable) { } - virtual float value() { - float result = (float)_callable.call().toNumber(); - return result; + virtual float peek() const { + return (float)const_cast(this)->_callable.call().toNumber(); } virtual void apply(float newValue, const Pointer& source) { diff --git a/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.cpp index d9b4a5fc59..bb9517b136 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.cpp @@ -12,8 +12,8 @@ using namespace controller; -float ScriptEndpoint::value() { - updateValue(); +float ScriptEndpoint::peek() const { + const_cast(this)->updateValue(); return _lastValueRead; } diff --git a/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.h b/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.h index 37160fcb48..836af721f6 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.h +++ b/libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.h @@ -24,8 +24,8 @@ public: : Endpoint(Input::INVALID_INPUT), _callable(callable) { } - virtual float value(); - virtual void apply(float newValue, const Pointer& source); + virtual float peek() const override; + virtual void apply(float newValue, const Pointer& source) override; protected: Q_INVOKABLE void updateValue(); diff --git a/libraries/controllers/src/controllers/impl/endpoints/StandardEndpoint.cpp b/libraries/controllers/src/controllers/impl/endpoints/StandardEndpoint.cpp index 09920d249c..89bbe5d777 100644 --- a/libraries/controllers/src/controllers/impl/endpoints/StandardEndpoint.cpp +++ b/libraries/controllers/src/controllers/impl/endpoints/StandardEndpoint.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "StandardEndpoint.h" \ No newline at end of file +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "StandardEndpoint.h" \ No newline at end of file diff --git a/libraries/controllers/src/controllers/impl/filters/ConstrainToIntegerFilter.cpp b/libraries/controllers/src/controllers/impl/filters/ConstrainToIntegerFilter.cpp index 78ffb47693..8bd3d2db89 100644 --- a/libraries/controllers/src/controllers/impl/filters/ConstrainToIntegerFilter.cpp +++ b/libraries/controllers/src/controllers/impl/filters/ConstrainToIntegerFilter.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "ConstrainToIntegerFilter.h" +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "ConstrainToIntegerFilter.h" diff --git a/libraries/controllers/src/controllers/impl/filters/ConstrainToPositiveIntegerFilter.cpp b/libraries/controllers/src/controllers/impl/filters/ConstrainToPositiveIntegerFilter.cpp index d78942b18f..f1abc8cecd 100644 --- a/libraries/controllers/src/controllers/impl/filters/ConstrainToPositiveIntegerFilter.cpp +++ b/libraries/controllers/src/controllers/impl/filters/ConstrainToPositiveIntegerFilter.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "ConstrainToPositiveIntegerFilter.h" +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "ConstrainToPositiveIntegerFilter.h" diff --git a/libraries/controllers/src/controllers/impl/filters/InvertFilter.cpp b/libraries/controllers/src/controllers/impl/filters/InvertFilter.cpp index db582b84cc..5407c6dd1d 100644 --- a/libraries/controllers/src/controllers/impl/filters/InvertFilter.cpp +++ b/libraries/controllers/src/controllers/impl/filters/InvertFilter.cpp @@ -6,4 +6,10 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#include "InvertFilter.h" +// NOTE: we don't need to include this header unless/until we add additional symbols. +// By removing this header we prevent these warnings on windows: +// +// warning LNK4221: This object file does not define any previously undefined public symbols, +// so it will not be used by any link operation that consumes this library +// +//#include "InvertFilter.h" diff --git a/libraries/display-plugins/CMakeLists.txt b/libraries/display-plugins/CMakeLists.txt index 14aa03de44..fad244fa5f 100644 --- a/libraries/display-plugins/CMakeLists.txt +++ b/libraries/display-plugins/CMakeLists.txt @@ -1,6 +1,6 @@ set(TARGET_NAME display-plugins) setup_hifi_library(OpenGL) -link_hifi_libraries(shared plugins gpu gl) +link_hifi_libraries(shared plugins gl) target_opengl() @@ -8,11 +8,6 @@ GroupSources("src/display-plugins") target_oglplus() -add_dependency_external_projects(LibOVR) -find_package(LibOVR REQUIRED) -target_include_directories(${TARGET_NAME} PRIVATE ${LIBOVR_INCLUDE_DIRS}) -target_link_libraries(${TARGET_NAME} ${LIBOVR_LIBRARIES}) - if (WIN32) add_dependency_external_projects(OpenVR) find_package(OpenVR REQUIRED) diff --git a/libraries/display-plugins/src/display-plugins/AbstractHMDScriptingInterface.cpp b/libraries/display-plugins/src/display-plugins/AbstractHMDScriptingInterface.cpp index 9987ae345c..4b8d957e5f 100644 --- a/libraries/display-plugins/src/display-plugins/AbstractHMDScriptingInterface.cpp +++ b/libraries/display-plugins/src/display-plugins/AbstractHMDScriptingInterface.cpp @@ -12,7 +12,6 @@ #include "DisplayPlugin.h" #include -#include static Setting::Handle IPD_SCALE_HANDLE("hmd.ipdScale", 1.0f); @@ -26,12 +25,12 @@ float AbstractHMDScriptingInterface::getIPD() const { float AbstractHMDScriptingInterface::getEyeHeight() const { // FIXME update the display plugin interface to expose per-plugin settings - return OVR_DEFAULT_EYE_HEIGHT; + return getPlayerHeight() - 0.10f; } float AbstractHMDScriptingInterface::getPlayerHeight() const { // FIXME update the display plugin interface to expose per-plugin settings - return OVR_DEFAULT_PLAYER_HEIGHT; + return 1.755f; } float AbstractHMDScriptingInterface::getIPDScale() const { diff --git a/libraries/display-plugins/src/display-plugins/Basic2DWindowOpenGLDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/Basic2DWindowOpenGLDisplayPlugin.cpp index 914d30d58a..9366ec4403 100644 --- a/libraries/display-plugins/src/display-plugins/Basic2DWindowOpenGLDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/Basic2DWindowOpenGLDisplayPlugin.cpp @@ -35,36 +35,36 @@ QAction* _vsyncAction{ nullptr }; void Basic2DWindowOpenGLDisplayPlugin::activate() { _framerateActions.clear(); - CONTAINER->addMenuItem(MENU_PATH(), FULLSCREEN, + _container->addMenuItem(MENU_PATH(), FULLSCREEN, [this](bool clicked) { if (clicked) { - CONTAINER->setFullscreen(getFullscreenTarget()); + _container->setFullscreen(getFullscreenTarget()); } else { - CONTAINER->unsetFullscreen(); + _container->unsetFullscreen(); } }, true, false); - CONTAINER->addMenu(FRAMERATE); + _container->addMenu(FRAMERATE); _framerateActions.push_back( - CONTAINER->addMenuItem(FRAMERATE, FRAMERATE_UNLIMITED, + _container->addMenuItem(FRAMERATE, FRAMERATE_UNLIMITED, [this](bool) { updateFramerate(); }, true, true, FRAMERATE)); _framerateActions.push_back( - CONTAINER->addMenuItem(FRAMERATE, FRAMERATE_60, + _container->addMenuItem(FRAMERATE, FRAMERATE_60, [this](bool) { updateFramerate(); }, true, false, FRAMERATE)); _framerateActions.push_back( - CONTAINER->addMenuItem(FRAMERATE, FRAMERATE_50, + _container->addMenuItem(FRAMERATE, FRAMERATE_50, [this](bool) { updateFramerate(); }, true, false, FRAMERATE)); _framerateActions.push_back( - CONTAINER->addMenuItem(FRAMERATE, FRAMERATE_40, + _container->addMenuItem(FRAMERATE, FRAMERATE_40, [this](bool) { updateFramerate(); }, true, false, FRAMERATE)); _framerateActions.push_back( - CONTAINER->addMenuItem(FRAMERATE, FRAMERATE_30, + _container->addMenuItem(FRAMERATE, FRAMERATE_30, [this](bool) { updateFramerate(); }, true, false, FRAMERATE)); WindowOpenGLDisplayPlugin::activate(); // Vsync detection happens in the parent class activate, so we need to check after that if (_vsyncSupported) { - _vsyncAction = CONTAINER->addMenuItem(MENU_PATH(), VSYNC_ON, [this](bool) {}, true, true); + _vsyncAction = _container->addMenuItem(MENU_PATH(), VSYNC_ON, [this](bool) {}, true, true); } else { _vsyncAction = nullptr; } @@ -107,7 +107,7 @@ int Basic2DWindowOpenGLDisplayPlugin::getDesiredInterval() const { bool Basic2DWindowOpenGLDisplayPlugin::isThrottled() const { static const QString ThrottleFPSIfNotFocus = "Throttle FPS If Not Focus"; // FIXME - this value duplicated in Menu.h - bool shouldThrottle = (!CONTAINER->isForeground() && CONTAINER->isOptionChecked(ThrottleFPSIfNotFocus)); + bool shouldThrottle = (!_container->isForeground() && _container->isOptionChecked(ThrottleFPSIfNotFocus)); if (_isThrottled != shouldThrottle) { _isThrottled = shouldThrottle; diff --git a/libraries/display-plugins/src/display-plugins/DisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/DisplayPlugin.cpp index 4af45d299b..8155d69826 100644 --- a/libraries/display-plugins/src/display-plugins/DisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/DisplayPlugin.cpp @@ -15,9 +15,6 @@ #include "Basic2DWindowOpenGLDisplayPlugin.h" #include "openvr/OpenVrDisplayPlugin.h" -#include "oculus/OculusDisplayPlugin.h" -#include "oculus/OculusDebugDisplayPlugin.h" -#include "oculus/OculusLegacyDisplayPlugin.h" const QString& DisplayPlugin::MENU_PATH() { static const QString value = "Display"; @@ -40,15 +37,6 @@ DisplayPluginList getDisplayPlugins() { new InterleavedStereoDisplayPlugin(), // HMDs - - // Windows Oculus SDK - new OculusDisplayPlugin(), - // Windows Oculus Simulator... uses head tracking and the same rendering - // as the connected hardware, but without using the SDK to display to the - // Rift. Useful for debugging Rift performance with nSight. - new OculusDebugDisplayPlugin(), - // Mac/Linux Oculus SDK (0.5) - new OculusLegacyDisplayPlugin(), #ifdef Q_OS_WIN // SteamVR SDK new OpenVrDisplayPlugin(), diff --git a/libraries/display-plugins/src/display-plugins/DisplayPlugin.h b/libraries/display-plugins/src/display-plugins/DisplayPlugin.h index b2176e0bd1..84c6592c53 100644 --- a/libraries/display-plugins/src/display-plugins/DisplayPlugin.h +++ b/libraries/display-plugins/src/display-plugins/DisplayPlugin.h @@ -5,135 +5,6 @@ // Distributed under the Apache License, Version 2.0. // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // -#pragma once -#include "plugins/Plugin.h" -#include -#include -#include - -#include -#include - -#include -#include -#include - -enum Eye { - Left, - Right, - Mono -}; - -/* - * Helper method to iterate over each eye - */ -template -void for_each_eye(F f) { - f(Left); - f(Right); -} - -/* - * Helper method to iterate over each eye, with an additional lambda to take action between the eyes - */ -template -void for_each_eye(F f, FF ff) { - f(Eye::Left); - ff(); - f(Eye::Right); -} - -class QWindow; - -#define AVERAGE_HUMAN_IPD 0.064f - -class DisplayPlugin : public Plugin { - Q_OBJECT -public: - virtual bool isHmd() const { return false; } - virtual int getHmdScreen() const { return -1; } - /// By default, all HMDs are stereo - virtual bool isStereo() const { return isHmd(); } - virtual bool isThrottled() const { return false; } - - // Rendering support - - // Stop requesting renders, but don't do full deactivation - // needed to work around the issues caused by Oculus - // processing messages in the middle of submitFrame - virtual void stop() = 0; - - /** - * Called by the application before the frame rendering. Can be used for - * render timing related calls (for instance, the Oculus begin frame timing - * call) - */ - virtual void preRender() = 0; - /** - * Called by the application immediately before calling the display function. - * For OpenGL based plugins, this is the best place to put activate the output - * OpenGL context - */ - virtual void preDisplay() = 0; - - /** - * Sends the scene texture to the display plugin. - */ - virtual void display(GLuint sceneTexture, const glm::uvec2& sceneSize) = 0; - - /** - * Called by the application immeidately after display. For OpenGL based - * displays, this is the best place to put the buffer swap - */ - virtual void finishFrame() = 0; - - // Does the rendering surface have current focus? - virtual bool hasFocus() const = 0; - - // The size of the rendering target (may be larger than the device size due to distortion) - virtual glm::uvec2 getRecommendedRenderSize() const = 0; - - // The size of the UI - virtual glm::uvec2 getRecommendedUiSize() const { - return getRecommendedRenderSize(); - } - - // By default the aspect ratio is just the render size - virtual float getRecommendedAspectRatio() const { - return aspect(getRecommendedRenderSize()); - } - - // Stereo specific methods - virtual glm::mat4 getProjection(Eye eye, const glm::mat4& baseProjection) const { - return baseProjection; - } - - // HMD specific methods - // TODO move these into another class? - virtual glm::mat4 getEyeToHeadTransform(Eye eye) const { - static const glm::mat4 transform; return transform; - } - - virtual glm::mat4 getHeadPose() const { - static const glm::mat4 pose; return pose; - } - - // Needed for timewarp style features - virtual void setEyeRenderPose(Eye eye, const glm::mat4& pose) { - // NOOP - } - - virtual float getIPD() const { return AVERAGE_HUMAN_IPD; } - - virtual void abandonCalibration() {} - virtual void resetSensors() {} - virtual float devicePixelRatio() { return 1.0; } - - - static const QString& MENU_PATH(); -signals: - void recommendedFramebufferSizeChanged(const QSize & size); - void requestRender(); -}; +#include diff --git a/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.cpp index 914f80d983..ce512962ff 100644 --- a/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.cpp @@ -25,6 +25,6 @@ bool NullDisplayPlugin::hasFocus() const { void NullDisplayPlugin::preRender() {} void NullDisplayPlugin::preDisplay() {} -void NullDisplayPlugin::display(GLuint sceneTexture, const glm::uvec2& sceneSize) {} +void NullDisplayPlugin::display(uint32_t sceneTexture, const glm::uvec2& sceneSize) {} void NullDisplayPlugin::finishFrame() {} void NullDisplayPlugin::stop() {} diff --git a/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.h b/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.h index 4f2cc77b8f..8cd5c2bc37 100644 --- a/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.h +++ b/libraries/display-plugins/src/display-plugins/NullDisplayPlugin.h @@ -21,7 +21,7 @@ public: virtual bool hasFocus() const override; virtual void preRender() override; virtual void preDisplay() override; - virtual void display(GLuint sceneTexture, const glm::uvec2& sceneSize) override; + virtual void display(uint32_t sceneTexture, const glm::uvec2& sceneSize) override; virtual void finishFrame() override; private: diff --git a/libraries/display-plugins/src/display-plugins/OpenGLDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/OpenGLDisplayPlugin.cpp index 3791375c9e..3ef882fe76 100644 --- a/libraries/display-plugins/src/display-plugins/OpenGLDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/OpenGLDisplayPlugin.cpp @@ -9,6 +9,7 @@ #include #include +#include #include #include diff --git a/libraries/display-plugins/src/display-plugins/WindowOpenGLDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/WindowOpenGLDisplayPlugin.cpp index ffea6605af..6ddc791503 100644 --- a/libraries/display-plugins/src/display-plugins/WindowOpenGLDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/WindowOpenGLDisplayPlugin.cpp @@ -42,7 +42,7 @@ bool WindowOpenGLDisplayPlugin::hasFocus() const { void WindowOpenGLDisplayPlugin::activate() { OpenGLDisplayPlugin::activate(); - _window = CONTAINER->getPrimarySurface(); + _window = _container->getPrimarySurface(); _window->makeCurrent(); customizeContext(); _window->doneCurrent(); diff --git a/libraries/display-plugins/src/display-plugins/openvr/OpenVrDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/openvr/OpenVrDisplayPlugin.cpp index 245fd11ef7..174bf1bf36 100644 --- a/libraries/display-plugins/src/display-plugins/openvr/OpenVrDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/openvr/OpenVrDisplayPlugin.cpp @@ -89,7 +89,7 @@ bool OpenVrDisplayPlugin::isSupported() const { } void OpenVrDisplayPlugin::activate() { - CONTAINER->setIsOptionChecked(StandingHMDSensorMode, true); + _container->setIsOptionChecked(StandingHMDSensorMode, true); hmdRefCount++; vr::HmdError eError = vr::HmdError_None; @@ -132,7 +132,7 @@ void OpenVrDisplayPlugin::activate() { } void OpenVrDisplayPlugin::deactivate() { - CONTAINER->setIsOptionChecked(StandingHMDSensorMode, false); + _container->setIsOptionChecked(StandingHMDSensorMode, false); hmdRefCount--; diff --git a/libraries/display-plugins/src/display-plugins/stereo/StereoDisplayPlugin.cpp b/libraries/display-plugins/src/display-plugins/stereo/StereoDisplayPlugin.cpp index 4f7b0a1a78..f7e71313df 100644 --- a/libraries/display-plugins/src/display-plugins/stereo/StereoDisplayPlugin.cpp +++ b/libraries/display-plugins/src/display-plugins/stereo/StereoDisplayPlugin.cpp @@ -74,21 +74,21 @@ void StereoDisplayPlugin::activate() { if (screen == qApp->primaryScreen()) { checked = true; } - auto action = CONTAINER->addMenuItem(MENU_PATH(), name, + auto action = _container->addMenuItem(MENU_PATH(), name, [this](bool clicked) { updateScreen(); }, true, checked, "Screens"); _screenActions[i] = action; } - CONTAINER->removeMenu(FRAMERATE); + _container->removeMenu(FRAMERATE); - CONTAINER->setFullscreen(qApp->primaryScreen()); + _container->setFullscreen(qApp->primaryScreen()); WindowOpenGLDisplayPlugin::activate(); } void StereoDisplayPlugin::updateScreen() { for (uint32_t i = 0; i < _screenActions.size(); ++i) { if (_screenActions[i]->isChecked()) { - CONTAINER->setFullscreen(qApp->screens().at(i)); + _container->setFullscreen(qApp->screens().at(i)); break; } } @@ -96,7 +96,7 @@ void StereoDisplayPlugin::updateScreen() { void StereoDisplayPlugin::deactivate() { _screenActions.clear(); - CONTAINER->unsetFullscreen(); + _container->unsetFullscreen(); WindowOpenGLDisplayPlugin::deactivate(); } diff --git a/libraries/input-plugins/src/input-plugins/InputPlugin.h b/libraries/input-plugins/src/input-plugins/InputPlugin.h index 787922e04c..d03f884ec7 100644 --- a/libraries/input-plugins/src/input-plugins/InputPlugin.h +++ b/libraries/input-plugins/src/input-plugins/InputPlugin.h @@ -10,14 +10,4 @@ // #pragma once -#include - -class InputPlugin : public Plugin { -public: - virtual bool isJointController() const = 0; - - virtual void pluginFocusOutEvent() = 0; - - virtual void pluginUpdate(float deltaTime, bool jointsCaptured) = 0; -}; - +#include diff --git a/libraries/input-plugins/src/input-plugins/KeyboardMouseDevice.cpp b/libraries/input-plugins/src/input-plugins/KeyboardMouseDevice.cpp index a0481dfaa0..9cd510c521 100755 --- a/libraries/input-plugins/src/input-plugins/KeyboardMouseDevice.cpp +++ b/libraries/input-plugins/src/input-plugins/KeyboardMouseDevice.cpp @@ -209,72 +209,3 @@ QString KeyboardMouseDevice::getDefaultMappingConfig() const { return MAPPING_JSON; } -//void KeyboardMouseDevice::assignDefaultInputMapping(UserInputMapper& mapper) { -// const float BUTTON_MOVE_SPEED = 1.0f; -// const float BUTTON_YAW_SPEED = 0.75f; -// const float BUTTON_PITCH_SPEED = 0.5f; -// const float MOUSE_YAW_SPEED = 0.5f; -// const float MOUSE_PITCH_SPEED = 0.25f; -// const float TOUCH_YAW_SPEED = 0.5f; -// const float TOUCH_PITCH_SPEED = 0.25f; -// const float BUTTON_BOOM_SPEED = 0.1f; -// -// // AWSD keys mapping -// -// mapper.addInputChannel(controller::BOOM_IN, makeInput(Qt::Key_E), makeInput(Qt::Key_Shift), BUTTON_BOOM_SPEED); -// mapper.addInputChannel(controller::BOOM_OUT, makeInput(Qt::Key_C), makeInput(Qt::Key_Shift), BUTTON_BOOM_SPEED); -// mapper.addInputChannel(controller::LATERAL_LEFT, makeInput(Qt::Key_A), makeInput(Qt::RightButton), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_RIGHT, makeInput(Qt::Key_D), makeInput(Qt::RightButton), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_LEFT, makeInput(Qt::Key_A), makeInput(Qt::Key_Shift), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_RIGHT, makeInput(Qt::Key_D), makeInput(Qt::Key_Shift), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::PITCH_DOWN, makeInput(Qt::Key_S), makeInput(Qt::Key_Shift), BUTTON_PITCH_SPEED); -// mapper.addInputChannel(controller::PITCH_UP, makeInput(Qt::Key_W), makeInput(Qt::Key_Shift), BUTTON_PITCH_SPEED); -// -// // Arrow keys mapping -// mapper.addInputChannel(controller::LONGITUDINAL_BACKWARD, makeInput(Qt::Key_Down), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(controller::LONGITUDINAL_FORWARD, makeInput(Qt::Key_Up), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(controller::YAW_LEFT, makeInput(Qt::Key_Left), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(controller::YAW_RIGHT, makeInput(Qt::Key_Right), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(controller::VERTICAL_DOWN, makeInput(Qt::Key_PageDown), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(controller::VERTICAL_UP, makeInput(Qt::Key_PageUp), BUTTON_MOVE_SPEED); -// -// mapper.addInputChannel(controller::LATERAL_LEFT, makeInput(Qt::Key_Left), makeInput(Qt::RightButton), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_RIGHT, makeInput(Qt::Key_Right), makeInput(Qt::RightButton), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_LEFT, makeInput(Qt::Key_Left), makeInput(Qt::Key_Shift), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_RIGHT, makeInput(Qt::Key_Right), makeInput(Qt::Key_Shift), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::PITCH_DOWN, makeInput(Qt::Key_Down), makeInput(Qt::Key_Shift), BUTTON_PITCH_SPEED); -// mapper.addInputChannel(controller::PITCH_UP, makeInput(Qt::Key_Up), makeInput(Qt::Key_Shift), BUTTON_PITCH_SPEED); -// -// // Mouse move -// mapper.addInputChannel(controller::PITCH_DOWN, makeInput(MOUSE_AXIS_Y_NEG), makeInput(Qt::RightButton), MOUSE_PITCH_SPEED); -// mapper.addInputChannel(controller::PITCH_UP, makeInput(MOUSE_AXIS_Y_POS), makeInput(Qt::RightButton), MOUSE_PITCH_SPEED); -// mapper.addInputChannel(controller::YAW_LEFT, makeInput(MOUSE_AXIS_X_NEG), makeInput(Qt::RightButton), MOUSE_YAW_SPEED); -// mapper.addInputChannel(controller::YAW_RIGHT, makeInput(MOUSE_AXIS_X_POS), makeInput(Qt::RightButton), MOUSE_YAW_SPEED); -// -// -//#ifdef Q_OS_MAC -// // wheel event modifier on Mac collide with the touchpad scroll event -// mapper.addInputChannel(controller::PITCH_DOWN, makeInput(TOUCH_AXIS_Y_NEG), TOUCH_PITCH_SPEED); -// mapper.addInputChannel(controller::PITCH_UP, makeInput(TOUCH_AXIS_Y_POS), TOUCH_PITCH_SPEED); -// mapper.addInputChannel(controller::YAW_LEFT, makeInput(TOUCH_AXIS_X_NEG), TOUCH_YAW_SPEED); -// mapper.addInputChannel(controller::YAW_RIGHT, makeInput(TOUCH_AXIS_X_POS), TOUCH_YAW_SPEED); -//#else -// // Touch pad yaw pitch -// mapper.addInputChannel(controller::PITCH_DOWN, makeInput(TOUCH_AXIS_Y_NEG), TOUCH_PITCH_SPEED); -// mapper.addInputChannel(controller::PITCH_UP, makeInput(TOUCH_AXIS_Y_POS), TOUCH_PITCH_SPEED); -// mapper.addInputChannel(controller::YAW_LEFT, makeInput(TOUCH_AXIS_X_NEG), TOUCH_YAW_SPEED); -// mapper.addInputChannel(controller::YAW_RIGHT, makeInput(TOUCH_AXIS_X_POS), TOUCH_YAW_SPEED); -// -// // Wheel move -// mapper.addInputChannel(controller::BOOM_IN, makeInput(MOUSE_AXIS_WHEEL_Y_POS), BUTTON_BOOM_SPEED); -// mapper.addInputChannel(controller::BOOM_OUT, makeInput(MOUSE_AXIS_WHEEL_Y_NEG), BUTTON_BOOM_SPEED); -// mapper.addInputChannel(controller::LATERAL_LEFT, makeInput(MOUSE_AXIS_WHEEL_X_NEG), BUTTON_YAW_SPEED); -// mapper.addInputChannel(controller::LATERAL_RIGHT, makeInput(MOUSE_AXIS_WHEEL_X_POS), BUTTON_YAW_SPEED); -// -//#endif -// -// mapper.addInputChannel(controller::SHIFT, makeInput(Qt::Key_Space)); -// mapper.addInputChannel(controller::ACTION1, makeInput(Qt::Key_R)); -// mapper.addInputChannel(controller::ACTION2, makeInput(Qt::Key_T)); -//} - diff --git a/libraries/input-plugins/src/input-plugins/SDL2Manager.cpp b/libraries/input-plugins/src/input-plugins/SDL2Manager.cpp index ec2fa2ed07..c6f5491a5a 100644 --- a/libraries/input-plugins/src/input-plugins/SDL2Manager.cpp +++ b/libraries/input-plugins/src/input-plugins/SDL2Manager.cpp @@ -20,27 +20,27 @@ #ifdef HAVE_SDL2 static_assert( - controller::A == SDL_CONTROLLER_BUTTON_A && - controller::B == SDL_CONTROLLER_BUTTON_B && - controller::X == SDL_CONTROLLER_BUTTON_X && - controller::Y == SDL_CONTROLLER_BUTTON_Y && - controller::BACK == SDL_CONTROLLER_BUTTON_BACK && - controller::GUIDE == SDL_CONTROLLER_BUTTON_GUIDE && - controller::START == SDL_CONTROLLER_BUTTON_START && - controller::LS == SDL_CONTROLLER_BUTTON_LEFTSTICK && - controller::RS == SDL_CONTROLLER_BUTTON_RIGHTSTICK && - controller::LB == SDL_CONTROLLER_BUTTON_LEFTSHOULDER && - controller::RB == SDL_CONTROLLER_BUTTON_RIGHTSHOULDER && - controller::DU == SDL_CONTROLLER_BUTTON_DPAD_UP && - controller::DD == SDL_CONTROLLER_BUTTON_DPAD_DOWN && - controller::DL == SDL_CONTROLLER_BUTTON_DPAD_LEFT && - controller::DR == SDL_CONTROLLER_BUTTON_DPAD_RIGHT && - controller::LX == SDL_CONTROLLER_AXIS_LEFTX && - controller::LY == SDL_CONTROLLER_AXIS_LEFTY && - controller::RX == SDL_CONTROLLER_AXIS_RIGHTX && - controller::RY == SDL_CONTROLLER_AXIS_RIGHTY && - controller::LT == SDL_CONTROLLER_AXIS_TRIGGERLEFT && - controller::RT == SDL_CONTROLLER_AXIS_TRIGGERRIGHT, + (int)controller::A == (int)SDL_CONTROLLER_BUTTON_A && + (int)controller::B == (int)SDL_CONTROLLER_BUTTON_B && + (int)controller::X == (int)SDL_CONTROLLER_BUTTON_X && + (int)controller::Y == (int)SDL_CONTROLLER_BUTTON_Y && + (int)controller::BACK == (int)SDL_CONTROLLER_BUTTON_BACK && + (int)controller::GUIDE == (int)SDL_CONTROLLER_BUTTON_GUIDE && + (int)controller::START == (int)SDL_CONTROLLER_BUTTON_START && + (int)controller::LS == (int)SDL_CONTROLLER_BUTTON_LEFTSTICK && + (int)controller::RS == (int)SDL_CONTROLLER_BUTTON_RIGHTSTICK && + (int)controller::LB == (int)SDL_CONTROLLER_BUTTON_LEFTSHOULDER && + (int)controller::RB == (int)SDL_CONTROLLER_BUTTON_RIGHTSHOULDER && + (int)controller::DU == (int)SDL_CONTROLLER_BUTTON_DPAD_UP && + (int)controller::DD == (int)SDL_CONTROLLER_BUTTON_DPAD_DOWN && + (int)controller::DL == (int)SDL_CONTROLLER_BUTTON_DPAD_LEFT && + (int)controller::DR == (int)SDL_CONTROLLER_BUTTON_DPAD_RIGHT && + (int)controller::LX == (int)SDL_CONTROLLER_AXIS_LEFTX && + (int)controller::LY == (int)SDL_CONTROLLER_AXIS_LEFTY && + (int)controller::RX == (int)SDL_CONTROLLER_AXIS_RIGHTX && + (int)controller::RY == (int)SDL_CONTROLLER_AXIS_RIGHTY && + (int)controller::LT == (int)SDL_CONTROLLER_AXIS_TRIGGERLEFT && + (int)controller::RT == (int)SDL_CONTROLLER_AXIS_TRIGGERRIGHT, "SDL2 equvalence: Enums and values from StandardControls.h are assumed to match enums from SDL_gamecontroller.h"); #endif diff --git a/libraries/input-plugins/src/input-plugins/SixenseManager.cpp b/libraries/input-plugins/src/input-plugins/SixenseManager.cpp index 6cb58ced82..024eb86182 100644 --- a/libraries/input-plugins/src/input-plugins/SixenseManager.cpp +++ b/libraries/input-plugins/src/input-plugins/SixenseManager.cpp @@ -94,8 +94,8 @@ void SixenseManager::activate() { _calibrationState = CALIBRATION_STATE_IDLE; _avatarPosition = DEFAULT_AVATAR_POSITION; - CONTAINER->addMenu(MENU_PATH); - CONTAINER->addMenuItem(MENU_PATH, TOGGLE_SMOOTH, + _container->addMenu(MENU_PATH); + _container->addMenuItem(MENU_PATH, TOGGLE_SMOOTH, [this] (bool clicked) { this->setSixenseFilter(clicked); }, true, true); @@ -136,8 +136,8 @@ void SixenseManager::deactivate() { InputPlugin::deactivate(); #ifdef HAVE_SIXENSE - CONTAINER->removeMenuItem(MENU_NAME, TOGGLE_SMOOTH); - CONTAINER->removeMenu(MENU_PATH); + _container->removeMenuItem(MENU_NAME, TOGGLE_SMOOTH); + _container->removeMenu(MENU_PATH); _poseStateMap.clear(); _collectedSamples.clear(); @@ -319,7 +319,7 @@ void SixenseManager::updateCalibration(void* controllersX) { _avatarRotation = glm::inverse(glm::quat_cast(glm::mat3(xAxis, Vectors::UNIT_Y, zAxis))); const float Y_OFFSET_CALIBRATED_HANDS_TO_AVATAR = -0.3f; _avatarPosition.y += Y_OFFSET_CALIBRATED_HANDS_TO_AVATAR; - CONTAINER->requestReset(); + _container->requestReset(); qCDebug(inputplugins, "succeess: sixense calibration"); } break; @@ -556,49 +556,6 @@ QString SixenseManager::getDefaultMappingConfig() const { return MAPPING_JSON; } -// -//void SixenseManager::assignDefaultInputMapping(UserInputMapper& mapper) { -// const float JOYSTICK_MOVE_SPEED = 1.0f; -// const float JOYSTICK_YAW_SPEED = 0.5f; -// const float JOYSTICK_PITCH_SPEED = 0.25f; -// const float BUTTON_MOVE_SPEED = 1.0f; -// const float BOOM_SPEED = 0.1f; -// using namespace controller; -// -// // Left Joystick: Movement, strafing -// mapper.addInputChannel(UserInputMapper::TRANSLATE_Z, makeInput(LY), JOYSTICK_MOVE_SPEED); -// mapper.addInputChannel(UserInputMapper::TRANSLATE_X, makeInput(LX), JOYSTICK_MOVE_SPEED); -// -// // Right Joystick: Camera orientation -// mapper.addInputChannel(UserInputMapper::YAW, makeInput(RX), JOYSTICK_YAW_SPEED); -// mapper.addInputChannel(UserInputMapper::PITCH, makeInput(RY), JOYSTICK_PITCH_SPEED); -// -// // Buttons -// mapper.addInputChannel(UserInputMapper::BOOM_IN, makeInput(L3), BOOM_SPEED); -// mapper.addInputChannel(UserInputMapper::BOOM_OUT, makeInput(L1), BOOM_SPEED); -// -// mapper.addInputChannel(UserInputMapper::VERTICAL_UP, makeInput(R3), BUTTON_MOVE_SPEED); -// mapper.addInputChannel(UserInputMapper::VERTICAL_DOWN, makeInput(R1), BUTTON_MOVE_SPEED); -// -// mapper.addInputChannel(UserInputMapper::SHIFT, makeInput(L2)); -// mapper.addInputChannel(UserInputMapper::SHIFT, makeInput(R2)); -// -// mapper.addInputChannel(UserInputMapper::ACTION1, makeInput(L4)); -// mapper.addInputChannel(UserInputMapper::ACTION2, makeInput(R4)); -// -// // FIXME -//// mapper.addInputChannel(UserInputMapper::LEFT_HAND, makeInput(LEFT_HAND)); -//// mapper.addInputChannel(UserInputMapper::RIGHT_HAND, makeInput(RIGHT_HAND)); -// -// mapper.addInputChannel(UserInputMapper::LEFT_HAND_CLICK, makeInput(LT)); -// mapper.addInputChannel(UserInputMapper::RIGHT_HAND_CLICK, makeInput(RT)); -// -// // TODO find a mechanism to allow users to navigate the context menu via -// mapper.addInputChannel(UserInputMapper::CONTEXT_MENU, makeInput(L0)); -// mapper.addInputChannel(UserInputMapper::TOGGLE_MUTE, makeInput(R0)); -// -//} - // virtual void SixenseManager::saveSettings() const { Settings settings; diff --git a/libraries/input-plugins/src/input-plugins/ViveControllerManager.cpp b/libraries/input-plugins/src/input-plugins/ViveControllerManager.cpp index e90006e014..69b2b5b2c6 100644 --- a/libraries/input-plugins/src/input-plugins/ViveControllerManager.cpp +++ b/libraries/input-plugins/src/input-plugins/ViveControllerManager.cpp @@ -75,8 +75,8 @@ bool ViveControllerManager::isSupported() const { void ViveControllerManager::activate() { InputPlugin::activate(); #ifdef Q_OS_WIN - CONTAINER->addMenu(MENU_PATH); - CONTAINER->addMenuItem(MENU_PATH, RENDER_CONTROLLERS, + _container->addMenu(MENU_PATH); + _container->addMenuItem(MENU_PATH, RENDER_CONTROLLERS, [this] (bool clicked) { this->setRenderControllers(clicked); }, true, true); @@ -146,8 +146,8 @@ void ViveControllerManager::deactivate() { InputPlugin::deactivate(); #ifdef Q_OS_WIN - CONTAINER->removeMenuItem(MENU_NAME, RENDER_CONTROLLERS); - CONTAINER->removeMenu(MENU_PATH); + _container->removeMenuItem(MENU_NAME, RENDER_CONTROLLERS); + _container->removeMenu(MENU_PATH); hmdRefCount--; diff --git a/libraries/plugins/src/plugins/DisplayPlugin.h b/libraries/plugins/src/plugins/DisplayPlugin.h new file mode 100644 index 0000000000..5b00391f09 --- /dev/null +++ b/libraries/plugins/src/plugins/DisplayPlugin.h @@ -0,0 +1,140 @@ +// +// Created by Bradley Austin Davis on 2015/05/29 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// +#pragma once + +#include + +#include +#include + +#include +#include + +#include +#include + +#include "Plugin.h" + +enum Eye { + Left, + Right, + Mono +}; + +/* + * Helper method to iterate over each eye + */ +template +void for_each_eye(F f) { + f(Left); + f(Right); +} + +/* + * Helper method to iterate over each eye, with an additional lambda to take action between the eyes + */ +template +void for_each_eye(F f, FF ff) { + f(Eye::Left); + ff(); + f(Eye::Right); +} + +class QWindow; + +#define AVERAGE_HUMAN_IPD 0.064f + +class DisplayPlugin : public Plugin { + Q_OBJECT +public: + virtual bool isHmd() const { return false; } + virtual int getHmdScreen() const { return -1; } + /// By default, all HMDs are stereo + virtual bool isStereo() const { return isHmd(); } + virtual bool isThrottled() const { return false; } + + // Rendering support + + // Stop requesting renders, but don't do full deactivation + // needed to work around the issues caused by Oculus + // processing messages in the middle of submitFrame + virtual void stop() = 0; + + /** + * Called by the application before the frame rendering. Can be used for + * render timing related calls (for instance, the Oculus begin frame timing + * call) + */ + virtual void preRender() = 0; + /** + * Called by the application immediately before calling the display function. + * For OpenGL based plugins, this is the best place to put activate the output + * OpenGL context + */ + virtual void preDisplay() = 0; + + /** + * Sends the scene texture to the display plugin. + */ + virtual void display(uint32_t sceneTexture, const glm::uvec2& sceneSize) = 0; + + /** + * Called by the application immeidately after display. For OpenGL based + * displays, this is the best place to put the buffer swap + */ + virtual void finishFrame() = 0; + + // Does the rendering surface have current focus? + virtual bool hasFocus() const = 0; + + // The size of the rendering target (may be larger than the device size due to distortion) + virtual glm::uvec2 getRecommendedRenderSize() const = 0; + + // The size of the UI + virtual glm::uvec2 getRecommendedUiSize() const { + return getRecommendedRenderSize(); + } + + // By default the aspect ratio is just the render size + virtual float getRecommendedAspectRatio() const { + return aspect(getRecommendedRenderSize()); + } + + // Stereo specific methods + virtual glm::mat4 getProjection(Eye eye, const glm::mat4& baseProjection) const { + return baseProjection; + } + + // HMD specific methods + // TODO move these into another class? + virtual glm::mat4 getEyeToHeadTransform(Eye eye) const { + static const glm::mat4 transform; return transform; + } + + virtual glm::mat4 getHeadPose() const { + static const glm::mat4 pose; return pose; + } + + // Needed for timewarp style features + virtual void setEyeRenderPose(Eye eye, const glm::mat4& pose) { + // NOOP + } + + virtual float getIPD() const { return AVERAGE_HUMAN_IPD; } + + virtual void abandonCalibration() {} + virtual void resetSensors() {} + virtual float devicePixelRatio() { return 1.0; } + + + static const QString& MENU_PATH(); +signals: + void recommendedFramebufferSizeChanged(const QSize & size); + void requestRender(); +}; + diff --git a/libraries/plugins/src/plugins/Forward.h b/libraries/plugins/src/plugins/Forward.h index 1a9298d13c..78ec8fdcb3 100644 --- a/libraries/plugins/src/plugins/Forward.h +++ b/libraries/plugins/src/plugins/Forward.h @@ -21,3 +21,4 @@ using DisplayPluginPointer = QSharedPointer; using DisplayPluginList = QVector; using InputPluginPointer = QSharedPointer; using InputPluginList = QVector; + diff --git a/libraries/plugins/src/plugins/InputPlugin.h b/libraries/plugins/src/plugins/InputPlugin.h new file mode 100644 index 0000000000..e9d8ac8d86 --- /dev/null +++ b/libraries/plugins/src/plugins/InputPlugin.h @@ -0,0 +1,23 @@ +// +// InputPlugin.h +// input-plugins/src/input-plugins +// +// Created by Sam Gondelman on 7/13/2015 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// +#pragma once + +#include "Plugin.h" + +class InputPlugin : public Plugin { +public: + virtual bool isJointController() const = 0; + + virtual void pluginFocusOutEvent() = 0; + + virtual void pluginUpdate(float deltaTime, bool jointsCaptured) = 0; +}; + diff --git a/libraries/plugins/src/plugins/Plugin.cpp b/libraries/plugins/src/plugins/Plugin.cpp index 2c0b9fa5cf..7c30f252c9 100644 --- a/libraries/plugins/src/plugins/Plugin.cpp +++ b/libraries/plugins/src/plugins/Plugin.cpp @@ -7,12 +7,10 @@ // #include "Plugin.h" -PluginContainer* Plugin::CONTAINER{ nullptr }; - QString Plugin::UNKNOWN_PLUGIN_ID("unknown"); void Plugin::setContainer(PluginContainer* container) { - CONTAINER = container; + _container = container; } bool Plugin::isSupported() const { return true; } diff --git a/libraries/plugins/src/plugins/Plugin.h b/libraries/plugins/src/plugins/Plugin.h index f53d309e97..c030b1073f 100644 --- a/libraries/plugins/src/plugins/Plugin.h +++ b/libraries/plugins/src/plugins/Plugin.h @@ -24,7 +24,7 @@ public: virtual bool isSupported() const; - static void setContainer(PluginContainer* container); + void setContainer(PluginContainer* container); /// Called when plugin is initially loaded, typically at application start virtual void init(); @@ -57,8 +57,8 @@ public: virtual void loadSettings() {} protected: - bool _active{ false }; - static PluginContainer* CONTAINER; + bool _active { false }; + PluginContainer* _container { nullptr }; static QString UNKNOWN_PLUGIN_ID; }; diff --git a/libraries/plugins/src/plugins/PluginManager.cpp b/libraries/plugins/src/plugins/PluginManager.cpp index 2deb41fb13..27e326fcba 100644 --- a/libraries/plugins/src/plugins/PluginManager.cpp +++ b/libraries/plugins/src/plugins/PluginManager.cpp @@ -6,15 +6,54 @@ // See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html // #include "PluginManager.h" + #include -#include "Forward.h" +#include +#include +#include +#include + +#include "RuntimePlugin.h" +#include "DisplayPlugin.h" +#include "InputPlugin.h" +#include "PluginContainer.h" + PluginManager* PluginManager::getInstance() { static PluginManager _manager; return &_manager; } +using Loader = QSharedPointer; +using LoaderList = QList; + +const LoaderList& getLoadedPlugins() { + static std::once_flag once; + static LoaderList loadedPlugins; + std::call_once(once, [&] { + QString pluginPath = QCoreApplication::applicationDirPath() + "/plugins/"; + QDir pluginDir(pluginPath); + pluginDir.setFilter(QDir::Files); + if (pluginDir.exists()) { + qDebug() << "Loading runtime plugins from " << pluginPath; + auto candidates = pluginDir.entryList(); + for (auto plugin : candidates) { + qDebug() << "Attempting plugins " << plugin; + QSharedPointer loader(new QPluginLoader(pluginPath + plugin)); + if (loader->load()) { + qDebug() << "Plugins " << plugin << " success"; + loadedPlugins.push_back(loader); + } + } + } + }); + return loadedPlugins; +} + +PluginManager::PluginManager() { +} + // TODO migrate to a DLL model where plugins are discovered and loaded at runtime by the PluginManager class extern DisplayPluginList getDisplayPlugins(); extern InputPluginList getInputPlugins(); @@ -23,8 +62,25 @@ extern void saveInputPluginSettings(const InputPluginList& plugins); const DisplayPluginList& PluginManager::getDisplayPlugins() { static DisplayPluginList displayPlugins; static std::once_flag once; + std::call_once(once, [&] { + // Grab the built in plugins displayPlugins = ::getDisplayPlugins(); + + // Now grab the dynamic plugins + for (auto loader : getLoadedPlugins()) { + DisplayProvider* displayProvider = qobject_cast(loader->instance()); + if (displayProvider) { + for (auto displayPlugin : displayProvider->getDisplayPlugins()) { + displayPlugins.push_back(displayPlugin); + } + } + } + auto& container = PluginContainer::getInstance(); + for (auto plugin : displayPlugins) { + plugin->setContainer(&container); + } + }); return displayPlugins; } @@ -34,6 +90,21 @@ const InputPluginList& PluginManager::getInputPlugins() { static std::once_flag once; std::call_once(once, [&] { inputPlugins = ::getInputPlugins(); + + // Now grab the dynamic plugins + for (auto loader : getLoadedPlugins()) { + InputProvider* inputProvider = qobject_cast(loader->instance()); + if (inputProvider) { + for (auto inputPlugin : inputProvider->getInputPlugins()) { + inputPlugins.push_back(inputPlugin); + } + } + } + + auto& container = PluginContainer::getInstance(); + for (auto plugin : inputPlugins) { + plugin->setContainer(&container); + } }); return inputPlugins; } diff --git a/libraries/plugins/src/plugins/PluginManager.h b/libraries/plugins/src/plugins/PluginManager.h index 09dcc9d107..17619a93c0 100644 --- a/libraries/plugins/src/plugins/PluginManager.h +++ b/libraries/plugins/src/plugins/PluginManager.h @@ -12,6 +12,7 @@ class PluginManager : public QObject { public: static PluginManager* getInstance(); + PluginManager(); const DisplayPluginList& getDisplayPlugins(); const InputPluginList& getInputPlugins(); diff --git a/libraries/plugins/src/plugins/RuntimePlugin.h b/libraries/plugins/src/plugins/RuntimePlugin.h new file mode 100644 index 0000000000..d7bf31ea28 --- /dev/null +++ b/libraries/plugins/src/plugins/RuntimePlugin.h @@ -0,0 +1,36 @@ +// +// Created by Bradley Austin Davis on 2015/10/24 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// +#pragma once + +#include + +#include +#include + +#include "Forward.h" + +class DisplayProvider { +public: + virtual ~DisplayProvider() {} + + virtual DisplayPluginList getDisplayPlugins() = 0; +}; + +#define DisplayProvider_iid "com.highfidelity.plugins.display" +Q_DECLARE_INTERFACE(DisplayProvider, DisplayProvider_iid) + + +class InputProvider { +public: + virtual ~InputProvider() {} + virtual InputPluginList getInputPlugins() = 0; +}; + +#define InputProvider_iid "com.highfidelity.plugins.input" +Q_DECLARE_INTERFACE(InputProvider, InputProvider_iid) + diff --git a/libraries/shared/CMakeLists.txt b/libraries/shared/CMakeLists.txt index d9df5eba7f..2691c51128 100644 --- a/libraries/shared/CMakeLists.txt +++ b/libraries/shared/CMakeLists.txt @@ -3,14 +3,4 @@ set(TARGET_NAME shared) # TODO: there isn't really a good reason to have Script linked here - let's get what is requiring it out (RegisteredMetaTypes.cpp) setup_hifi_library(Gui Network Script Widgets) -find_package(ZLIB REQUIRED) -target_link_libraries(${TARGET_NAME} ${ZLIB_LIBRARIES}) -target_include_directories(${TARGET_NAME} PUBLIC ${ZLIB_INCLUDE_DIRS}) - -if (WIN32) - # Birarda will fix this when he finds it. - get_filename_component(ZLIB_LIB_DIR "${ZLIB_LIBRARIES}" DIRECTORY) - get_filename_component(ZLIB_DIR "${ZLIB_LIB_DIR}" DIRECTORY) - set(ZLIB_BIN_DIR "${ZLIB_DIR}/bin") - add_paths_to_fixup_libs(${ZLIB_BIN_DIR}) -endif () +target_zlib() diff --git a/plugins/CMakeLists.txt b/plugins/CMakeLists.txt new file mode 100644 index 0000000000..55b18b122c --- /dev/null +++ b/plugins/CMakeLists.txt @@ -0,0 +1,18 @@ +# +# Created by Bradley Austin Davis on 2015/10/25 +# Copyright 2015 High Fidelity, Inc. +# +# Distributed under the Apache License, Version 2.0. +# See the accompanying file LICENSE or http:#www.apache.org/licenses/LICENSE-2.0.html +# + +# add the plugin directories +file(GLOB PLUGIN_SUBDIRS RELATIVE "${CMAKE_CURRENT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}/*") +list(REMOVE_ITEM PLUGIN_SUBDIRS "CMakeFiles") + +foreach(DIR ${PLUGIN_SUBDIRS}) + if(IS_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/${DIR}") + add_subdirectory(${DIR}) + endif() +endforeach() + diff --git a/plugins/oculus/CMakeLists.txt b/plugins/oculus/CMakeLists.txt new file mode 100644 index 0000000000..fe1a87d6b6 --- /dev/null +++ b/plugins/oculus/CMakeLists.txt @@ -0,0 +1,25 @@ +# +# Created by Bradley Austin Davis on 2015/10/25 +# Copyright 2015 High Fidelity, Inc. +# +# Distributed under the Apache License, Version 2.0. +# See the accompanying file LICENSE or http:#www.apache.org/licenses/LICENSE-2.0.html +# + +if (WIN32) + + # we're using static GLEW, so define GLEW_STATIC + add_definitions(-DGLEW_STATIC) + + set(TARGET_NAME oculus) + setup_hifi_plugin() + link_hifi_libraries(shared gl plugins display-plugins) + + include_hifi_library_headers(octree) + + add_dependency_external_projects(LibOVR) + find_package(LibOVR REQUIRED) + target_include_directories(${TARGET_NAME} PRIVATE ${LIBOVR_INCLUDE_DIRS}) + target_link_libraries(${TARGET_NAME} ${LIBOVR_LIBRARIES}) + +endif() \ No newline at end of file diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.cpp b/plugins/oculus/src/OculusBaseDisplayPlugin.cpp similarity index 95% rename from libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.cpp rename to plugins/oculus/src/OculusBaseDisplayPlugin.cpp index 859a4a810a..4c80b9a51d 100644 --- a/libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.cpp +++ b/plugins/oculus/src/OculusBaseDisplayPlugin.cpp @@ -61,6 +61,14 @@ bool OculusBaseDisplayPlugin::isSupported() const { #endif } +// DLL based display plugins MUST initialize GLEW inside the DLL code. +void OculusBaseDisplayPlugin::customizeContext() { + glewExperimental = true; + GLenum err = glewInit(); + glGetError(); + WindowOpenGLDisplayPlugin::customizeContext(); +} + void OculusBaseDisplayPlugin::init() { } diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.h b/plugins/oculus/src/OculusBaseDisplayPlugin.h similarity index 96% rename from libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.h rename to plugins/oculus/src/OculusBaseDisplayPlugin.h index 6307f6bbf9..ba1924bfff 100644 --- a/libraries/display-plugins/src/display-plugins/oculus/OculusBaseDisplayPlugin.h +++ b/plugins/oculus/src/OculusBaseDisplayPlugin.h @@ -7,7 +7,7 @@ // #pragma once -#include "../WindowOpenGLDisplayPlugin.h" +#include #include @@ -35,6 +35,7 @@ public: virtual float getIPD() const override final; protected: + virtual void customizeContext() override; virtual void preRender() override final; virtual void display(GLuint finalTexture, const glm::uvec2& sceneSize) override; diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusDebugDisplayPlugin.cpp b/plugins/oculus/src/OculusDebugDisplayPlugin.cpp similarity index 100% rename from libraries/display-plugins/src/display-plugins/oculus/OculusDebugDisplayPlugin.cpp rename to plugins/oculus/src/OculusDebugDisplayPlugin.cpp diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusDebugDisplayPlugin.h b/plugins/oculus/src/OculusDebugDisplayPlugin.h similarity index 100% rename from libraries/display-plugins/src/display-plugins/oculus/OculusDebugDisplayPlugin.h rename to plugins/oculus/src/OculusDebugDisplayPlugin.h diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusDisplayPlugin.cpp b/plugins/oculus/src/OculusDisplayPlugin.cpp similarity index 97% rename from libraries/display-plugins/src/display-plugins/oculus/OculusDisplayPlugin.cpp rename to plugins/oculus/src/OculusDisplayPlugin.cpp index 3e2290f104..923b8bde6e 100644 --- a/libraries/display-plugins/src/display-plugins/oculus/OculusDisplayPlugin.cpp +++ b/plugins/oculus/src/OculusDisplayPlugin.cpp @@ -7,11 +7,14 @@ // #include "OculusDisplayPlugin.h" -#include +#include + +// FIXME get rid of this +#include +#include #include "OculusHelpers.h" -#include #if (OVR_MAJOR_VERSION >= 6) @@ -142,16 +145,16 @@ static const QString FRAMERATE = DisplayPlugin::MENU_PATH() + ">Framerate"; void OculusDisplayPlugin::activate() { - CONTAINER->addMenuItem(MENU_PATH(), MONO_PREVIEW, + _container->addMenuItem(MENU_PATH(), MONO_PREVIEW, [this](bool clicked) { _monoPreview = clicked; }, true, true); - CONTAINER->removeMenu(FRAMERATE); + _container->removeMenu(FRAMERATE); OculusBaseDisplayPlugin::activate(); } void OculusDisplayPlugin::customizeContext() { - WindowOpenGLDisplayPlugin::customizeContext(); + OculusBaseDisplayPlugin::customizeContext(); #if (OVR_MAJOR_VERSION >= 6) _sceneFbo = SwapFboPtr(new SwapFramebufferWrapper(_hmd)); _sceneFbo->Init(getRecommendedRenderSize()); diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusDisplayPlugin.h b/plugins/oculus/src/OculusDisplayPlugin.h similarity index 100% rename from libraries/display-plugins/src/display-plugins/oculus/OculusDisplayPlugin.h rename to plugins/oculus/src/OculusDisplayPlugin.h diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusHelpers.cpp b/plugins/oculus/src/OculusHelpers.cpp similarity index 100% rename from libraries/display-plugins/src/display-plugins/oculus/OculusHelpers.cpp rename to plugins/oculus/src/OculusHelpers.cpp diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusHelpers.h b/plugins/oculus/src/OculusHelpers.h similarity index 100% rename from libraries/display-plugins/src/display-plugins/oculus/OculusHelpers.h rename to plugins/oculus/src/OculusHelpers.h diff --git a/plugins/oculus/src/OculusProvider.cpp b/plugins/oculus/src/OculusProvider.cpp new file mode 100644 index 0000000000..40dfb9df9a --- /dev/null +++ b/plugins/oculus/src/OculusProvider.cpp @@ -0,0 +1,54 @@ +// +// Created by Bradley Austin Davis on 2015/10/25 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// + +#include + +#include +#include +#include + +#include +#include + +#include "OculusDisplayPlugin.h" +#include "OculusDebugDisplayPlugin.h" + +class OculusProvider : public QObject, public DisplayProvider +{ + Q_OBJECT + Q_PLUGIN_METADATA(IID DisplayProvider_iid FILE "oculus.json") + Q_INTERFACES(DisplayProvider) + +public: + OculusProvider(QObject* parent = nullptr) : QObject(parent) {} + virtual ~OculusProvider() {} + + virtual DisplayPluginList getDisplayPlugins() override { + static std::once_flag once; + std::call_once(once, [&] { + DisplayPluginPointer plugin(new OculusDisplayPlugin()); + if (plugin->isSupported()) { + _displayPlugins.push_back(plugin); + } + + // Windows Oculus Simulator... uses head tracking and the same rendering + // as the connected hardware, but without using the SDK to display to the + // Rift. Useful for debugging Rift performance with nSight. + plugin = DisplayPluginPointer(new OculusDebugDisplayPlugin()); + if (plugin->isSupported()) { + _displayPlugins.push_back(plugin); + } + }); + return _displayPlugins; + } + +private: + DisplayPluginList _displayPlugins; +}; + +#include "OculusProvider.moc" diff --git a/plugins/oculus/src/oculus.json b/plugins/oculus/src/oculus.json new file mode 100644 index 0000000000..0967ef424b --- /dev/null +++ b/plugins/oculus/src/oculus.json @@ -0,0 +1 @@ +{} diff --git a/plugins/oculusLegacy/CMakeLists.txt b/plugins/oculusLegacy/CMakeLists.txt new file mode 100644 index 0000000000..bf9d22410d --- /dev/null +++ b/plugins/oculusLegacy/CMakeLists.txt @@ -0,0 +1,22 @@ +# +# Created by Bradley Austin Davis on 2015/10/25 +# Copyright 2015 High Fidelity, Inc. +# +# Distributed under the Apache License, Version 2.0. +# See the accompanying file LICENSE or http:#www.apache.org/licenses/LICENSE-2.0.html +# + +if (NOT WIN32) + + set(TARGET_NAME oculusLegacy) + setup_hifi_plugin() + link_hifi_libraries(shared gl plugins display-plugins) + + include_hifi_library_headers(octree) + + add_dependency_external_projects(LibOVR) + find_package(LibOVR REQUIRED) + target_include_directories(${TARGET_NAME} PRIVATE ${LIBOVR_INCLUDE_DIRS}) + target_link_libraries(${TARGET_NAME} ${LIBOVR_LIBRARIES}) + +endif() \ No newline at end of file diff --git a/plugins/oculusLegacy/src/OculusHelpers.cpp b/plugins/oculusLegacy/src/OculusHelpers.cpp new file mode 100644 index 0000000000..fff2a38344 --- /dev/null +++ b/plugins/oculusLegacy/src/OculusHelpers.cpp @@ -0,0 +1,9 @@ +// +// Created by Bradley Austin Davis on 2015/08/08 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// + +#include "OculusHelpers.h" diff --git a/plugins/oculusLegacy/src/OculusHelpers.h b/plugins/oculusLegacy/src/OculusHelpers.h new file mode 100644 index 0000000000..b4bcdc1511 --- /dev/null +++ b/plugins/oculusLegacy/src/OculusHelpers.h @@ -0,0 +1,85 @@ +// +// Created by Bradley Austin Davis on 2015/05/26 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// +#pragma once + +#include +#include +#include +#include + +// Convenience method for looping over each eye with a lambda +template +inline void ovr_for_each_eye(Function function) { + for (ovrEyeType eye = ovrEyeType::ovrEye_Left; + eye < ovrEyeType::ovrEye_Count; + eye = static_cast(eye + 1)) { + function(eye); + } +} + +inline glm::mat4 toGlm(const ovrMatrix4f & om) { + return glm::transpose(glm::make_mat4(&om.M[0][0])); +} + +inline glm::mat4 toGlm(const ovrFovPort & fovport, float nearPlane = 0.01f, float farPlane = 10000.0f) { + return toGlm(ovrMatrix4f_Projection(fovport, nearPlane, farPlane, true)); +} + +inline glm::vec3 toGlm(const ovrVector3f & ov) { + return glm::make_vec3(&ov.x); +} + +inline glm::vec2 toGlm(const ovrVector2f & ov) { + return glm::make_vec2(&ov.x); +} + +inline glm::uvec2 toGlm(const ovrSizei & ov) { + return glm::uvec2(ov.w, ov.h); +} + +inline glm::quat toGlm(const ovrQuatf & oq) { + return glm::make_quat(&oq.x); +} + +inline glm::mat4 toGlm(const ovrPosef & op) { + glm::mat4 orientation = glm::mat4_cast(toGlm(op.Orientation)); + glm::mat4 translation = glm::translate(glm::mat4(), toGlm(op.Position)); + return translation * orientation; +} + +inline ovrMatrix4f ovrFromGlm(const glm::mat4 & m) { + ovrMatrix4f result; + glm::mat4 transposed(glm::transpose(m)); + memcpy(result.M, &(transposed[0][0]), sizeof(float) * 16); + return result; +} + +inline ovrVector3f ovrFromGlm(const glm::vec3 & v) { + return{ v.x, v.y, v.z }; +} + +inline ovrVector2f ovrFromGlm(const glm::vec2 & v) { + return{ v.x, v.y }; +} + +inline ovrSizei ovrFromGlm(const glm::uvec2 & v) { + return{ (int)v.x, (int)v.y }; +} + +inline ovrQuatf ovrFromGlm(const glm::quat & q) { + return{ q.x, q.y, q.z, q.w }; +} + +inline ovrPosef ovrPoseFromGlm(const glm::mat4 & m) { + glm::vec3 translation = glm::vec3(m[3]) / m[3].w; + glm::quat orientation = glm::quat_cast(m); + ovrPosef result; + result.Orientation = ovrFromGlm(orientation); + result.Position = ovrFromGlm(translation); + return result; +} diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.cpp b/plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.cpp similarity index 90% rename from libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.cpp rename to plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.cpp index 205444397f..d8dc3667ae 100644 --- a/libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.cpp +++ b/plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.cpp @@ -42,10 +42,8 @@ uvec2 OculusLegacyDisplayPlugin::getRecommendedRenderSize() const { } void OculusLegacyDisplayPlugin::preRender() { -#if (OVR_MAJOR_VERSION == 5) ovrHmd_GetEyePoses(_hmd, _frameIndex, _eyeOffsets, _eyePoses, &_trackingState); ovrHmd_BeginFrame(_hmd, _frameIndex); -#endif WindowOpenGLDisplayPlugin::preRender(); } @@ -54,32 +52,21 @@ glm::mat4 OculusLegacyDisplayPlugin::getProjection(Eye eye, const glm::mat4& bas } void OculusLegacyDisplayPlugin::resetSensors() { -#if (OVR_MAJOR_VERSION == 5) ovrHmd_RecenterPose(_hmd); -#endif } glm::mat4 OculusLegacyDisplayPlugin::getEyeToHeadTransform(Eye eye) const { -#if (OVR_MAJOR_VERSION == 5) return toGlm(_eyePoses[eye]); -#else - return WindowOpenGLDisplayPlugin::getEyeToHeadTransform(eye); -#endif } // Should NOT be used for rendering as this will mess up timewarp. Use the getModelview() method above for // any use of head poses for rendering, ensuring you use the correct eye glm::mat4 OculusLegacyDisplayPlugin::getHeadPose() const { -#if (OVR_MAJOR_VERSION == 5) return toGlm(_trackingState.HeadPose.ThePose); -#else - return WindowOpenGLDisplayPlugin::getHeadPose(); -#endif } bool OculusLegacyDisplayPlugin::isSupported() const { -#if (OVR_MAJOR_VERSION == 5) if (!ovr_Initialize(nullptr)) { return false; } @@ -104,14 +91,10 @@ bool OculusLegacyDisplayPlugin::isSupported() const { ovr_Shutdown(); return result; -#else - return false; -#endif } void OculusLegacyDisplayPlugin::activate() { -#if (OVR_MAJOR_VERSION == 5) - if (!OVR_SUCCESS(ovr_Initialize(nullptr))) { + if (!(ovr_Initialize(nullptr))) { Q_ASSERT(false); qFatal("Failed to Initialize SDK"); } @@ -149,8 +132,8 @@ void OculusLegacyDisplayPlugin::activate() { _frameIndex = 0; - if (!OVR_SUCCESS(ovrHmd_ConfigureTracking(_hmd, - ovrTrackingCap_Orientation | ovrTrackingCap_Position | ovrTrackingCap_MagYawCorrection, 0))) { + if (!ovrHmd_ConfigureTracking(_hmd, + ovrTrackingCap_Orientation | ovrTrackingCap_Position | ovrTrackingCap_MagYawCorrection, 0)) { qFatal("Could not attach to sensor device"); } @@ -158,7 +141,7 @@ void OculusLegacyDisplayPlugin::activate() { int screen = getHmdScreen(); if (screen != -1) { - CONTAINER->setFullscreen(qApp->screens()[screen]); + _container->setFullscreen(qApp->screens()[screen]); } _window->installEventFilter(this); @@ -189,11 +172,9 @@ void OculusLegacyDisplayPlugin::activate() { #endif ovrHmd_ConfigureRendering(_hmd, &config.Config, distortionCaps, _eyeFovs, _eyeRenderDescs); Q_ASSERT(result); -#endif } void OculusLegacyDisplayPlugin::deactivate() { -#if (OVR_MAJOR_VERSION == 5) _window->removeEventFilter(this); WindowOpenGLDisplayPlugin::deactivate(); @@ -202,12 +183,19 @@ void OculusLegacyDisplayPlugin::deactivate() { if (_hmdScreen >= 0) { riftScreen = qApp->screens()[_hmdScreen]; } - CONTAINER->unsetFullscreen(riftScreen); + _container->unsetFullscreen(riftScreen); ovrHmd_Destroy(_hmd); _hmd = nullptr; ovr_Shutdown(); -#endif +} + +// DLL based display plugins MUST initialize GLEW inside the DLL code. +void OculusLegacyDisplayPlugin::customizeContext() { + glewExperimental = true; + glewInit(); + glGetError(); + WindowOpenGLDisplayPlugin::customizeContext(); } void OculusLegacyDisplayPlugin::preDisplay() { @@ -216,17 +204,14 @@ void OculusLegacyDisplayPlugin::preDisplay() { void OculusLegacyDisplayPlugin::display(GLuint finalTexture, const glm::uvec2& sceneSize) { ++_frameIndex; -#if (OVR_MAJOR_VERSION == 5) ovr_for_each_eye([&](ovrEyeType eye) { reinterpret_cast(_eyeTextures[eye]).OGL.TexId = finalTexture; }); ovrHmd_EndFrame(_hmd, _eyePoses, _eyeTextures); -#endif } // Pass input events on to the application bool OculusLegacyDisplayPlugin::eventFilter(QObject* receiver, QEvent* event) { -#if (OVR_MAJOR_VERSION == 5) if (!_hswDismissed && (event->type() == QEvent::KeyPress)) { static ovrHSWDisplayState hswState; ovrHmd_GetHSWDisplayState(_hmd, &hswState); @@ -236,7 +221,6 @@ bool OculusLegacyDisplayPlugin::eventFilter(QObject* receiver, QEvent* event) { _hswDismissed = true; } } -#endif return WindowOpenGLDisplayPlugin::eventFilter(receiver, event); } diff --git a/libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.h b/plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.h similarity index 95% rename from libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.h rename to plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.h index 9e2e47f434..6e3f864aee 100644 --- a/libraries/display-plugins/src/display-plugins/oculus/OculusLegacyDisplayPlugin.h +++ b/plugins/oculusLegacy/src/OculusLegacyDisplayPlugin.h @@ -7,7 +7,7 @@ // #pragma once -#include "../WindowOpenGLDisplayPlugin.h" +#include #include @@ -35,6 +35,7 @@ public: virtual glm::mat4 getHeadPose() const override; protected: + virtual void customizeContext() override; virtual void preRender() override; virtual void preDisplay() override; virtual void display(GLuint finalTexture, const glm::uvec2& sceneSize) override; diff --git a/plugins/oculusLegacy/src/OculusProvider.cpp b/plugins/oculusLegacy/src/OculusProvider.cpp new file mode 100644 index 0000000000..606563e0ad --- /dev/null +++ b/plugins/oculusLegacy/src/OculusProvider.cpp @@ -0,0 +1,45 @@ +// +// Created by Bradley Austin Davis on 2015/10/25 +// Copyright 2015 High Fidelity, Inc. +// +// Distributed under the Apache License, Version 2.0. +// See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html +// + +#include + +#include +#include +#include + +#include +#include + +#include "OculusLegacyDisplayPlugin.h" + +class OculusProvider : public QObject, public DisplayProvider +{ + Q_OBJECT + Q_PLUGIN_METADATA(IID DisplayProvider_iid FILE "oculus.json") + Q_INTERFACES(DisplayProvider) + +public: + OculusProvider(QObject* parent = nullptr) : QObject(parent) {} + virtual ~OculusProvider() {} + + virtual DisplayPluginList getDisplayPlugins() override { + static std::once_flag once; + std::call_once(once, [&] { + DisplayPluginPointer plugin(new OculusLegacyDisplayPlugin()); + if (plugin->isSupported()) { + _displayPlugins.push_back(plugin); + } + }); + return _displayPlugins; + } + +private: + DisplayPluginList _displayPlugins; +}; + +#include "OculusProvider.moc" diff --git a/plugins/oculusLegacy/src/oculus.json b/plugins/oculusLegacy/src/oculus.json new file mode 100644 index 0000000000..0967ef424b --- /dev/null +++ b/plugins/oculusLegacy/src/oculus.json @@ -0,0 +1 @@ +{} diff --git a/tests/controllers/src/main.cpp b/tests/controllers/src/main.cpp index a7b1be15ca..139d9b282c 100644 --- a/tests/controllers/src/main.cpp +++ b/tests/controllers/src/main.cpp @@ -80,9 +80,6 @@ using namespace controller; class PluginContainerProxy : public QObject, PluginContainer { Q_OBJECT public: - PluginContainerProxy() { - Plugin::setContainer(this); - } virtual ~PluginContainerProxy() {} virtual void addMenu(const QString& menuName) override {} virtual void removeMenu(const QString& menuName) override {}