Merge branch 'main' into simplify_cxx_example

Author: darbyjohnston

CMakeLists.txt

@@ -34,6 +34,7 @@ option(OTIO_INSTALL_COMMANDLINE_TOOLS "Install the OTIO command line tools" ON)
 option(OTIO_INSTALL_CONTRIB           "Install the opentimelineio_contrib Python package" ON)
 set(OTIO_IMATH_LIBS "" CACHE STRING   "Imath library overrides to use instead of src/deps or find_package")
 option(OTIO_FIND_IMATH                "Find Imath using find_package, ignored if OTIO_IMATH_LIBS is set" OFF)
+option(OTIO_FIND_RAPIDJSON            "Find RapidJSON using find_package" OFF)
 set(OTIO_PYTHON_INSTALL_DIR "" CACHE STRING "Python installation dir (such as the site-packages dir)")
 
 # Build options
@@ -223,13 +224,10 @@ if (NOT "${OTIO_IMATH_LIBS}" STREQUAL "")
     set(OTIO_RESOLVED_IMATH_LIBRARIES "${OTIO_IMATH_LIBS}")
     set(USE_DEPS_IMATH OFF)
 elseif(OTIO_FIND_IMATH)
-    find_package(Imath QUIET)
+    set(USE_DEPS_IMATH OFF)
+    find_package(Imath REQUIRED)
     if (Imath_FOUND)
         message(STATUS "Found Imath 3 at ${Imath_CONFIG}")
-        set(USE_DEPS_IMATH OFF)
-    else()
-        message(STATUS "Imath 3 was not found, using src/deps/Imath")
-        set(USE_DEPS_IMATH ON)
     endif()
 else()
     message(STATUS "Using src/deps/Imath by default")
@@ -240,6 +238,17 @@ if(USE_DEPS_IMATH)
     include_directories("${PROJECT_SOURCE_DIR}/src/deps/Imath/src")
 endif()
 
+#----- RapidJSON
+
+if(OTIO_FIND_RAPIDJSON)
+    find_package(RapidJSON CONFIG REQUIRED)
+    if (RapidJSON_FOUND)
+        message(STATUS "Found RapidJSON at ${RapidJSON_CONFIG}")
+    endif()
+else()
+    message(STATUS "Using src/deps/rapidjson by default")
+endif()
+
 # set up the internally hosted dependencies
 add_subdirectory(src/deps)
 

src/deps/CMakeLists.txt

@@ -4,7 +4,12 @@
 #----- Other dependencies
 
 # detect if the submodules haven't been updated
-set(DEPS_SUBMODULES pybind11 rapidjson)
+set(DEPS_SUBMODULES pybind11)
+
+if(NOT OTIO_FIND_RAPIDJSON)
+    set(DEPS_SUBMODULES ${DEPS_SUBMODULES} rapidjson)
+endif()
+
 foreach(submodule IN LISTS DEPS_SUBMODULES)
     file(GLOB SUBMOD_CONTENTS ${submodule})
     list(LENGTH SUBMOD_CONTENTS SUBMOD_CONTENT_LEN)

src/opentimelineio/CMakeLists.txt

@@ -78,9 +78,15 @@ add_library(opentimelineio ${OTIO_SHARED_OR_STATIC_LIB}
 add_library(OTIO::opentimelineio ALIAS opentimelineio)
 
 target_include_directories(opentimelineio
-    PRIVATE       "${PROJECT_SOURCE_DIR}/src"
-                  "${PROJECT_SOURCE_DIR}/src/deps"
-                  "${PROJECT_SOURCE_DIR}/src/deps/rapidjson/include")
+    PRIVATE "${PROJECT_SOURCE_DIR}/src")
+
+if(OTIO_FIND_RAPIDJSON)
+    target_include_directories(opentimelineio
+        PRIVATE "${RapidJSON_INCLUDE_DIRS}")
+else()
+    target_include_directories(opentimelineio
+        PRIVATE "${PROJECT_SOURCE_DIR}/src/deps/rapidjson/include")
+endif()
 
 
 target_link_libraries(opentimelineio 

src/opentimelineio/serialization.cpp

@@ -77,6 +77,7 @@ class Encoder
     virtual void write_value(class TimeTransform const& value)       = 0;
     virtual void write_value(struct SerializableObject::ReferenceId) = 0;
     virtual void write_value(IMATH_NAMESPACE::Box2d const&)          = 0;
+    virtual void write_value(IMATH_NAMESPACE::V2d const&)            = 0;
 
 protected:
     void _error(ErrorStatus const& error_status)
@@ -269,7 +270,7 @@ class CloningEncoder : public Encoder
         _store(std::any(value));
     }
 
-    void write_value(IMATH_NAMESPACE::V2d const& value)
+    void write_value(IMATH_NAMESPACE::V2d const& value) override
     {
 
         if (_result_object_policy == ResultObjectPolicy::OnlyAnyDictionary)

src/py-opentimelineio/opentimelineio/url_utils.py

@@ -77,7 +77,11 @@ def filepath_from_url(urlstr):
         filepath = pathlib.PurePosixPath(filepath.drive, *filepath.parts[1:])
 
     # If the specified index is a windows drive, then offset the path
-    elif pathlib.PureWindowsPath(filepath.parts[1]).drive:
+    elif (
+        # relative paths may not have a parts[1]
+        len(filepath.parts) > 1
+        and pathlib.PureWindowsPath(filepath.parts[1]).drive
+    ):
         # Remove leading "/" if/when `request.url2pathname` yields
         # "/S:/path/file.ext"
         filepath = pathlib.PurePosixPath(*filepath.parts[1:])

tests/test_url_conversions.py

@@ -79,6 +79,14 @@ def test_posix_urls(self):
             processed_url = otio.url_utils.filepath_from_url(url)
             self.assertEqual(processed_url, POSIX_PATH)
 
+    def test_relative_url(self):
+        # see github issue #1817 - when a relative URL has only one name after
+        # the "." (ie ./blah but not ./blah/blah)
+        self.assertEqual(
+            otio.url_utils.filepath_from_url(os.path.join(".", "docs")),
+            "docs",
+        )
+
 
 if __name__ == "__main__":
     unittest.main()

tests/test_v2d.py

@@ -1,12 +1,14 @@
 # SPDX-License-Identifier: Apache-2.0
 # Copyright Contributors to the OpenTimelineIO project
 
-import unittest
+import json
 import sys
+import unittest
 
-import opentimelineio as otio
 import opentimelineio.test_utils as otio_test_utils
 
+import opentimelineio as otio
+
 
 class V2dTests(unittest.TestCase, otio_test_utils.OTIOAssertions):
     def test_cons(self):
@@ -21,15 +23,9 @@ def test_cons(self):
     def test_str(self):
         v = otio.schema.V2d(1.0, 2.0)
 
-        self.assertMultiLineEqual(
-            str(v),
-            'V2d(1.0, 2.0)'
-        )
+        self.assertMultiLineEqual(str(v), "V2d(1.0, 2.0)")
 
-        self.assertMultiLineEqual(
-            repr(v),
-            'otio.schema.V2d(x=1.0, y=2.0)'
-        )
+        self.assertMultiLineEqual(repr(v), "otio.schema.V2d(x=1.0, y=2.0)")
 
     def test_equality(self):
         v1 = otio.schema.V2d(1.0, 2.0)
@@ -104,6 +100,17 @@ def test_limits(self):
         self.assertEqual(otio.schema.V2d.baseTypeSmallest(), sys.float_info.min)
         self.assertEqual(otio.schema.V2d.baseTypeEpsilon(), sys.float_info.epsilon)
 
+    def test_json_serialization(self):
+        serialized = otio.adapters.otio_json.write_to_string(otio.schema.V2d())
+        json_v2d = json.loads(serialized)
+        self.assertEqual(json_v2d["OTIO_SCHEMA"], "V2d.1")
+
+    def test_serialization_round_trip(self):
+        v2d = otio.schema.V2d()
+        serialized = otio.adapters.otio_json.write_to_string(v2d)
+        deserialized = otio.adapters.otio_json.read_from_string(serialized)
+        self.assertEqual(v2d, deserialized)
+
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     unittest.main()