pye57 0.4.15__tar.gz → 0.4.16__tar.gz

{pye57-0.4.15/src/pye57.egg-info → pye57-0.4.16}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pye57
-Version: 0.4.15
+Version: 0.4.16
 Summary: Python .e57 files reader/writer
 Home-page: https://www.github.com/davidcaron/pye57
 Author: David Caron
@@ -11,14 +11,13 @@ License: MIT
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Programming Language :: Python :: Implementation :: CPython
-Requires-Python: >=3.8
+Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: numpy

{pye57-0.4.15 → pye57-0.4.16}/setup.py

@@ -90,12 +90,14 @@ class BuildExt(build_ext):
             opts.append("-DCRCPP_USE_CPP11")
             opts.append("-DCRCPP_BRANCHLESS")
             opts.append("-Wno-unused-variable")
+            opts.append("-DE57_ENABLE_DIAGNOSTIC_OUTPUT")
         elif ct == "msvc":
             opts.append(f'/DVERSION_INFO="{version}"')
             opts.append(rf'/DREVISION_ID="\"{revision_id}\""')
             opts.append("/DCRCPP_USE_CPP11")
             opts.append("/DCRCPP_BRANCHLESS")
             opts.append("/DWINDOWS")
+            opts.append("/DE57_ENABLE_DIAGNOSTIC_OUTPUT")
         for ext in self.extensions:
             ext.extra_compile_args = opts
 
@@ -132,7 +134,6 @@ setup(
         "License :: OSI Approved :: MIT License",
         "Programming Language :: Python",
         "Programming Language :: Python :: 3",
-        "Programming Language :: Python :: 3.8",
         "Programming Language :: Python :: 3.9",
         "Programming Language :: Python :: 3.10",
         "Programming Language :: Python :: 3.11",
@@ -142,5 +143,5 @@ setup(
     ],
     cmdclass={"build_ext": BuildExt},
     zip_safe=False,
-    python_requires=">=3.8",
+    python_requires=">=3.9",
 )

pye57-0.4.16/src/pye57/__version__.py

@@ -0,0 +1 @@
+__version__ = "0.4.16"

{pye57-0.4.15 → pye57-0.4.16}/src/pye57/e57.py

@@ -376,9 +376,9 @@ class E57:
             max_row = np.max(data["rowIndex"])
             min_col = np.min(data["columnIndex"])
             max_col = np.max(data["columnIndex"])
-            points_prototype.set("rowIndex", libe57.IntegerNode(self.image_file, 0, min_row, max_row))
+            points_prototype.set("rowIndex", libe57.IntegerNode(self.image_file, min_row, min_row, max_row))
             field_names.append("rowIndex")
-            points_prototype.set("columnIndex", libe57.IntegerNode(self.image_file, 0, min_col, max_col))
+            points_prototype.set("columnIndex", libe57.IntegerNode(self.image_file, min_col, min_col, max_col))
             field_names.append("columnIndex")
 
         if "cartesianInvalidState" in data:

{pye57-0.4.15 → pye57-0.4.16}/src/pye57/libe57_wrapper.cpp

@@ -7,6 +7,8 @@
 #include <E57Format.h>
 #include <E57Version.h>
 #include <ASTMVersion.h>
+#include <sstream>
+#include <string.h>
 
 namespace py = pybind11;
 using namespace pybind11::literals;
@@ -40,11 +42,14 @@ PYBIND11_MODULE(libe57, m) {
 
     static py::exception<E57Exception> exc(m, "E57Exception");
     py::register_exception_translator([](std::exception_ptr p) {
-    try {
-        if (p) std::rethrow_exception(p);
-    } catch (const E57Exception &e) {
-        exc(Utilities::errorCodeToString(e.errorCode()).c_str());
-    }
+        try {
+            if (p) std::rethrow_exception(p);
+        } catch (const E57Exception &e) {
+            std::stringstream ss;
+            e.report(__FILE__, __LINE__, __FUNCTION__, ss);
+            std::string output = e.errorStr() + std::string("\n\n") + ss.str();
+            exc(output.c_str());
+        }
     });
 
     m.attr("E57_FORMAT_MAJOR") = E57_FORMAT_MAJOR;

{pye57-0.4.15 → pye57-0.4.16/src/pye57.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pye57
-Version: 0.4.15
+Version: 0.4.16
 Summary: Python .e57 files reader/writer
 Home-page: https://www.github.com/davidcaron/pye57
 Author: David Caron
@@ -11,14 +11,13 @@ License: MIT
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Programming Language :: Python :: Implementation :: CPython
-Requires-Python: >=3.8
+Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: numpy

{pye57-0.4.15 → pye57-0.4.16}/tests/test_main.py

@@ -419,3 +419,24 @@ def test_clone_e57(e57_with_data_and_images_path, temp_e57_write):
 
     in_image.close()
     out_image.close()
+
+
+def test_write_e57_with_rowindex_and_columnindex_omiting_low_values(temp_e57_write):
+
+    with pye57.E57(temp_e57_write, mode='w') as e57:
+        # set some test points with missing row and column 0 (so np.min of it in write_scan_raw is larger than 0)
+        data_raw = {}
+        data_raw["cartesianX"] = np.array([0, 1, 2, 3]).astype(float)
+        data_raw["cartesianY"] = np.array([0, 1, 2, 3]).astype(float)
+        data_raw["cartesianZ"] = np.array([0, 1, 2, 3]).astype(float)
+        data_raw["rowIndex"] = np.array([1, 1, 2, 3])
+        data_raw["columnIndex"] = np.array([1, 1, 2, 3])
+
+        try:
+            # the next line will throw without the suggested fix
+            e57.write_scan_raw(data_raw, name='test_output_with_row_and_column')
+        except pye57.libe57.E57Exception as ex:
+            print(ex)
+            assert False
+    
+    assert os.path.isfile(temp_e57_write)

pye57-0.4.15/src/pye57/__version__.py

@@ -1 +0,0 @@
-__version__ = "0.4.15"