Merge pull request #173 from meerk40t/issue-172

Fix Issue #172, .github directory
meerk40t · Mar 22, 2022 · 7a7f419 · 7a7f419
2 parents 177cf99 + c6b50c5
commit 7a7f419
Show file tree

Hide file tree

Showing 13 changed files with 19 additions and 288 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
diff --git a/.github/CODE_OF_CONDUCT.md b/.github/CODE_OF_CONDUCT.md
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
diff --git a/.github/FUNDING.yml b/.github/FUNDING.yml
@@ -0,0 +1 @@
+github: [tatarize]
diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md
diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml
diff --git a/.github/ISSUE_TEMPLATE/config.yml b/.github/ISSUE_TEMPLATE/config.yml
diff --git a/.github/labeler.yml b/.github/labeler.yml
diff --git a/.github/workflows/pull-request-labeler.yml b/.github/workflows/pull-request-labeler.yml
diff --git a/.github/workflows/unittests.yml b/.github/workflows/unittests.yml
@@ -55,7 +55,7 @@ jobs:
       uses: actions/cache@v2
       with:
         path: ${{ env.pythonLocation }}
-        key: ${{ matrix.os }}-python-${{ steps.full-python-version.outputs.version }}
+        key: new-${{ matrix.os }}-python-${{ steps.full-python-version.outputs.version }}
 
     - name: Install Python dependencies
       run: |

diff --git a/setup.cfg b/setup.cfg
@@ -1,6 +1,6 @@
 [metadata]
 name = svgelements
-version = 1.6.10
+version = 1.6.11
 description = Svg Elements Parsing
 long_description_content_type=text/markdown
 long_description = file: README.md

diff --git a/svgelements/svgelements.py b/svgelements/svgelements.py
@@ -43,7 +43,7 @@
 and the Arc can do exact arc calculations if scipy is installed.
 """
 
-SVGELEMENTS_VERSION = "1.6.10"
+SVGELEMENTS_VERSION = "1.6.11"
 
 MIN_DEPTH = 5
 ERROR = 1e-12
@@ -4573,7 +4573,7 @@ def _abs_derivative(t):
             return abs(
                 3 * (p1 - p0) * (1 - t) ** 2
                 + 6 * (p2 - p1) * (1 - t) * t
-                + 3 * (p3 - p2) * t ** 2
+                + 3 * (p3 - p2) * t**2
             )
 
         return quad(_abs_derivative, 0.0, 1.0, epsabs=error, limit=1000)[0]
@@ -7113,7 +7113,7 @@ def _init_points(self, points):
                 self.points = list()
                 return
         try:
-            if len(points) == 1:
+            if len(points) == 1 and isinstance(points[0], (list, tuple, str)):
                 points = points[0]
         except TypeError:
             pass

diff --git a/test/test_shape.py b/test/test_shape.py
@@ -553,6 +553,19 @@ def test_issue_104(self):
         m = SVG.parse(q)
         self.assertEqual(len(m), 0)
 
+    def test_issue_172(self):
+        """
+        Polylines were checked whether the payload had a length of one and replaced them. However that was wrong
+        as when the payload was *actually* 1 it made it a point and then did not produce the single point.
+        """
+        p = Polyline(0,0)
+        q = Polyline(p)
+        self.assertEqual(len(q), 1)
+
+        p = Polygon(0, 0)
+        q = Polygon(p)
+        self.assertEqual(len(q), 1)
+
     def test_rect_strict(self):
         values = {
             'tag': 'rect',