From a1eb2ed990e1022328d568da2e1637a8b6e5b209 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B8rgen=20Lind?= Date: Fri, 19 Mar 2021 12:59:31 +0100 Subject: [PATCH 1/2] Remove VectorWrapper from python api --- python/openvds/PyGlobal.cpp | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/python/openvds/PyGlobal.cpp b/python/openvds/PyGlobal.cpp index 8f6cf4c1..0b0ba0f1 100644 --- a/python/openvds/PyGlobal.cpp +++ b/python/openvds/PyGlobal.cpp @@ -236,10 +236,6 @@ PyGlobal::initModule(py::module& m) m.def("isCompressionMethodSupported", static_cast(&IsCompressionMethodSupported), py::arg("compressionMethod").none(false), py::call_guard(), OPENVDS_DOCSTRING(IsCompressionMethodSupported)); m.def("open" , static_cast(&Open), py::arg("options").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Open_3)); m.def("open" , static_cast(&Open), py::arg("ioManager").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Open_4)); - m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_5)); - m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::Error &)>(&Create), py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_6)); - m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("ioManager").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_7)); - m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::Error &)>(&Create), py::arg("ioManager").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_8)); m.def("getLayout" , static_cast(&GetLayout), py::arg("handle").none(false), py::call_guard(), OPENVDS_DOCSTRING(GetLayout)); m.def("getAccessManagerInterface" , static_cast(&GetAccessManagerInterface), py::arg("handle").none(false), py::call_guard(), OPENVDS_DOCSTRING(GetAccessManagerInterface)); m.def("getAccessManager" , static_cast(&GetAccessManager), py::arg("handle").none(false), py::call_guard(), OPENVDS_DOCSTRING(GetAccessManager)); @@ -274,6 +270,10 @@ PyGlobal::initModule(py::module& m) // IMPLEMENTED : m.def("openWithAdaptiveCompressionTolerance", static_cast(&OpenWithAdaptiveCompressionTolerance), py::arg("url").none(false), py::arg("connectionString").none(false), py::arg("waveletAdaptiveTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(OpenWithAdaptiveCompressionTolerance)); // IMPLEMENTED : m.def("openWithAdaptiveCompressionRatio", static_cast(&OpenWithAdaptiveCompressionRatio), py::arg("url").none(false), py::arg("connectionString").none(false), py::arg("waveletAdaptiveRatio").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(OpenWithAdaptiveCompressionRatio)); // IMPLEMENTED : m.def("open" , static_cast(&Open), py::arg("url").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Open_2)); + // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_5)); + // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::Error &)>(&Create), py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_6)); + // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("ioManager").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_7)); + // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::Error &)>(&Create), py::arg("ioManager").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_8)); // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("url").none(false), py::arg("connectionString").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create)); // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::Error &)>(&Create), py::arg("url").none(false), py::arg("connectionString").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_2)); // IMPLEMENTED : m.def("create" , static_cast, VectorWrapper, const native::MetadataReadAccess &, native::CompressionMethod, float, native::Error &)>(&Create), py::arg("url").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_3)); @@ -393,6 +393,21 @@ PyGlobal::initModule(py::module& m) } return handle; }, py::arg("url").none(false), py::arg("waveletAdaptiveRatio").none(false), OPENVDS_DOCSTRING(OpenWithAdaptiveCompressionRatio)); + m.def("create" , [](const native::OpenOptions &options, const native::VolumeDataLayoutDescriptor &layoutDescriptor, const std::vector &axisDescriptors, const std::vector &channelDescriptors, const native::MetadataReadAccess &metadata, native::CompressionMethod compressionMethod, float tolerance, native::Error &error) + { + return native::Create(options, layoutDescriptor, axisDescriptors, channelDescriptors, metadata, compressionMethod, tolerance, error); + + }, py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_5)); + m.def("create" , [](const native::OpenOptions &options, const native::VolumeDataLayoutDescriptor &layoutDescriptor, const std::vector &axisDescriptors, const std::vector &channelDescriptors, const native::MetadataReadAccess &metadata, native::CompressionMethod compressionMethod, float tolerance) + { + native::Error err; + auto handle = native::Create(options, layoutDescriptor, axisDescriptors, channelDescriptors, metadata, compressionMethod, tolerance, err); + if (err.code) + { + throw std::runtime_error(err.string); + } + return handle; + }, py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_5)); m.def("create" , [](const std::string &url, const std::string &connectionString, const native::VolumeDataLayoutDescriptor &layout, std::vector axisdesc, std::vector channeldesc, const native::MetadataReadAccess &metadata, native::Error &error) { return native::Create(url, connectionString, layout , axisdesc, channeldesc, metadata, error); @@ -421,6 +436,10 @@ PyGlobal::initModule(py::module& m) } return handle; }, py::arg("url").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), OPENVDS_DOCSTRING(Create_2)); + m.def("create", [](const native::OpenOptions& opt, const native::VolumeDataLayoutDescriptor& layout, std::vector axisdesc, std::vector channeldesc, const native::MetadataReadAccess& metadata, native::Error& error) + { + return native::Create(opt, layout , axisdesc, channeldesc, metadata, error); + }, py::arg("options").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create_6)); m.def("create" , [](const native::OpenOptions &opt, const native::VolumeDataLayoutDescriptor &layout, std::vector axisdesc, std::vector channeldesc, const native::MetadataReadAccess &metadata) { native::Error err; @@ -441,11 +460,11 @@ PyGlobal::initModule(py::module& m) } return handle; }, py::arg("ioManager").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), OPENVDS_DOCSTRING(Create_4)); - m.def("create", [](const std::string& url, const std::string connectionString, const native::VolumeDataLayoutDescriptor& layout, VectorWrapper axisdesc, VectorWrapper channel, const native::MetadataReadAccess& metadata, native::CompressionMethod compression, float tolerance, native::Error& error) + m.def("create", [](const std::string& url, const std::string connectionString, const native::VolumeDataLayoutDescriptor& layout, std::vector axisdesc, std::vector channel, const native::MetadataReadAccess& metadata, native::CompressionMethod compression, float tolerance, native::Error& error) { return native::Create(url, connectionString, layout, axisdesc, channel, metadata, compression, tolerance, error); }, py::arg("url").none(false), py::arg("connectionString").none(false), py::arg("layoutDescriptor").none(false), py::arg("axisDescriptors").none(false), py::arg("channelDescriptors").none(false), py::arg("metadata").none(false), py::arg("compressionMethod").none(false), py::arg("compressionTolerance").none(false), py::arg("error").none(false), py::call_guard(), OPENVDS_DOCSTRING(Create)); - m.def("create", [](const std::string& url, const std::string connectionString, const native::VolumeDataLayoutDescriptor& layout, VectorWrapper axisdesc, VectorWrapper channel, const native::MetadataReadAccess& metadata, native::CompressionMethod compression, float tolerance) + m.def("create", [](const std::string& url, const std::string connectionString, const native::VolumeDataLayoutDescriptor& layout, std::vector axisdesc, std::vector channel, const native::MetadataReadAccess& metadata, native::CompressionMethod compression, float tolerance) { native::Error error; auto handle = native::Create(url, connectionString, layout, axisdesc, channel, metadata, compression, tolerance, error); -- GitLab From d6786a78999b56633c36557440c5d5aeff536e57 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B8rgen=20Lind?= Date: Fri, 19 Mar 2021 13:04:51 +0100 Subject: [PATCH 2/2] Mark release 2.0.2 --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index f0b0ead9..80667bd7 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -5,7 +5,7 @@ if(CCACHE_PROGRAM) set_property(GLOBAL PROPERTY RULE_LAUNCH_COMPILE "${CCACHE_PROGRAM}") endif() -project(OpenVDS VERSION 2.0.1) +project(OpenVDS VERSION 2.0.2) set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_CURRENT_SOURCE_DIR}/CMake") -- GitLab