1.0.beta.12

.github/workflows/codeball.yml

@@ -0,0 +1,12 @@
+name: Codeball
+on: [pull_request]
+
+jobs:
+  codeball_job:
+    runs-on: ubuntu-latest
+    name: Codeball
+    steps:
+      # Run Codeball on all new Pull Requests 🚀
+      # For customizations and more documentation, see https://github.com/sturdy-dev/codeball-action
+      - name: Codeball
+        uses: sturdy-dev/codeball-action@v2

.github/workflows/codeql-analysis.yml

@@ -39,11 +39,11 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
-        uses: github/codeql-action/init@v1
+        uses: github/codeql-action/init@v2
         with:
           languages: ${{ matrix.language }}
           # If you wish to specify custom queries, you can do so here or in a config file.
@@ -54,7 +54,7 @@ jobs:
       # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
       # If this step fails, then you should remove it and run the build manually (see below)
       - name: Autobuild
-        uses: github/codeql-action/autobuild@v1
+        uses: github/codeql-action/autobuild@v2
 
       # ℹ️ Command-line programs to run using the OS shell.
       # 📚 https://git.io/JvXDl
@@ -68,5 +68,5 @@ jobs:
       #   make release
 
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v1
+        uses: github/codeql-action/analyze@v2
 ...

.github/workflows/codesee-arch-diagram.yml

@@ -0,0 +1,81 @@
+on:
+  push:
+    branches:
+      - develop
+  pull_request_target:
+    types: [opened, synchronize, reopened]
+
+name: CodeSee Map
+
+jobs:
+  test_map_action:
+    runs-on: ubuntu-latest
+    continue-on-error: true
+    name: Run CodeSee Map Analysis
+    steps:
+      - name: checkout
+        id: checkout
+        uses: actions/checkout@v3
+        with:
+          repository: ${{ github.event.pull_request.head.repo.full_name }}
+          ref: ${{ github.event.pull_request.head.ref }}
+          fetch-depth: 0
+
+      # codesee-detect-languages has an output with id languages.
+      - name: Detect Languages
+        id: detect-languages
+        uses: Codesee-io/codesee-detect-languages-action@latest
+
+      - name: Configure JDK 16
+        uses: actions/setup-java@v3
+        if: ${{ fromJSON(steps.detect-languages.outputs.languages).java }}
+        with:
+          java-version: '16'
+          distribution: 'zulu'
+
+      # CodeSee Maps Go support uses a static binary so there's no setup step required.
+
+      - name: Configure Node.js 14
+        uses: actions/setup-node@v3.4.1
+        if: ${{ fromJSON(steps.detect-languages.outputs.languages).javascript }}
+        with:
+          node-version: '14'
+
+      - name: Configure Python 3.x
+        uses: actions/setup-python@v4
+        if: ${{ fromJSON(steps.detect-languages.outputs.languages).python }}
+        with:
+          python-version: '3.10'
+          architecture: 'x64'
+
+      - name: Configure Ruby '3.x'
+        uses: ruby/setup-ruby@v1
+        if: ${{ fromJSON(steps.detect-languages.outputs.languages).ruby }}
+        with:
+          ruby-version: '3.0'
+
+      # CodeSee Maps Rust support uses a static binary so there's no setup step required.
+
+      - name: Generate Map
+        id: generate-map
+        uses: Codesee-io/codesee-map-action@latest
+        with:
+          step: map
+          github_ref: ${{ github.ref }}
+          languages: ${{ steps.detect-languages.outputs.languages }}
+
+      - name: Upload Map
+        id: upload-map
+        uses: Codesee-io/codesee-map-action@latest
+        with:
+          step: mapUpload
+          api_token: ${{ secrets.CODESEE_ARCH_DIAG_API_TOKEN }}
+          github_ref: ${{ github.ref }}
+
+      - name: Insights
+        id: insights
+        uses: Codesee-io/codesee-map-action@latest
+        with:
+          step: insights
+          api_token: ${{ secrets.CODESEE_ARCH_DIAG_API_TOKEN }}
+          github_ref: ${{ github.ref }}

.github/workflows/run-all-tests.yml

@@ -9,12 +9,12 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ['3.6', '3.7', '3.8', '3.9', '3.10']
+        python-version: ['3.7', '3.8', '3.9', '3.10', '3.11-dev']
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v2
+        uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies
@@ -26,7 +26,7 @@ jobs:
           pytest pygeoif --cov=pygeoif --cov-fail-under=100 --cov-report=xml
       - name: "Upload coverage to Codecov"
         if: ${{ matrix.python-version==3.9 }}
-        uses: codecov/codecov-action@v2
+        uses: codecov/codecov-action@v3
         with:
           fail_ci_if_error: true
           verbose: true
@@ -38,9 +38,9 @@ jobs:
         python-version: ['3.9']
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v2
+        uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies
@@ -65,11 +65,11 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        pypy-version: ['pypy-3.6', 'pypy-3.7', 'pypy-3.8']
+        pypy-version: ['pypy-3.7', 'pypy-3.8', 'pypy-3.9']
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: Set up Python ${{ matrix.pypy-version }}
-        uses: actions/setup-python@v2
+        uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.pypy-version }}
       - name: Install dependencies
@@ -86,9 +86,9 @@ jobs:
     name: Build and publish to PyPI and TestPyPI
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@master
+      - uses: actions/checkout@v3
       - name: Set up Python 3.9
-        uses: actions/setup-python@v1
+        uses: actions/setup-python@v4
         with:
           python-version: 3.9
       - name: Install pypa/build

.gitignore

@@ -10,3 +10,4 @@ dist/
 venv/
 coverage.xml
 .mutmut-cache
+html/

.pre-commit-config.yaml

@@ -1,7 +1,10 @@
 ---
 repos:
+  - repo: meta
+    hooks:
+      - id: check-useless-excludes
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.1.0
+    rev: v4.3.0
     hooks:
       - id: check-added-large-files
       - id: check-docstring-first
@@ -21,39 +24,40 @@ repos:
       - id: requirements-txt-fixer
       - id: trailing-whitespace
   - repo: https://github.com/psf/black
-    rev: 21.12b0
+    rev: 22.6.0
     hooks:
       - id: black
   - repo: https://github.com/PyCQA/flake8
-    rev: 4.0.1
+    rev: 5.0.4
     hooks:
       - id: flake8
         additional_dependencies:
           - flake8
           - flake8-absolute-import
           - flake8-awesome
+          - flake8-bandit
           - flake8-blind-except
           - flake8-bugbear
-          - flake8-class-attributes-order
+          # - flake8-class-attributes-order
           - flake8-cognitive-complexity
           - flake8-commas
           - flake8-comments
           - flake8-complex-f-strings
           - flake8-continuation
           - flake8-debugger
           - flake8-docstrings
-          - flake8-dunder-all
+          # - flake8-dunder-all
           - flake8-encodings
           - flake8-expression-complexity
-          - flake8-function-order
+          # - flake8-function-order
           - flake8-implicit-str-concat
           - flake8-noqa
           - flake8-pep3101
           - flake8-print
           - flake8-raise
           - flake8-rst-docstrings
           - flake8-simplify
-          - flake8-slots
+          # - flake8-slots
           - flake8-string-format
           - flake8-super
           - flake8-typing-imports
@@ -64,15 +68,15 @@ repos:
     hooks:
       - id: isort
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev:  v0.931
+    rev: v0.971
     hooks:
       - id: mypy
   - repo: https://github.com/Lucas-C/pre-commit-hooks-markup
     rev: v1.0.1
     hooks:
-    - id: rst-linter
+      - id: rst-linter
   - repo: https://github.com/mgedmin/check-manifest
-    rev: "0.47"
+    rev: "0.48"
     hooks:
-    - id: check-manifest
+      - id: check-manifest
 ...

README.rst

@@ -45,6 +45,10 @@ It was written to provide clean and python only geometries for fastkml_
    :target: https://www.codefactor.io/repository/github/cleder/pygeoif/overview/main
    :alt: CodeFactor
 
+.. image:: https://img.shields.io/badge/pre--commit-enabled-brightgreen?logo=pre-commit&logoColor=white
+   :target: https://github.com/pre-commit/pre-commit
+   :alt: pre-commit
+
 Example
 ========
 

docs/HISTORY.txt

@@ -1,18 +1,24 @@
 Changelog
 =========
 
-1.0.beta.1 (2021-09-20)
+1.0 (unreleased)
 ------------------------
 
 - Add type annotations
 - refactor
+- changes to keep functionality and interface close to ``shapely``
 - remove support for python 2
 - minimum python version is 3.6
 - rename as_shape to shape
 - add box factory
 - format with black
 - reconstruct objects from their representation
 - Parse WKT that is not in upper case.
+- Centroid for LinearRings
+- Convex Hull
+- implement equality ``__eq__`` operator (``==``)
+- ``is_empty`` and ``bool``
+- drop duplicate points when creating LineStrings
 
 0.7 (2017/05/04)
 -----------------

pygeoif/factories.py

@@ -67,7 +67,7 @@ def orient(polygon: Polygon, ccw: bool = True) -> Polygon:
     and the interiors will be in clockwise orientation, or
     the other way round when ccw is False.
     """
-    s = -1.0 if not ccw else 1.0
+    s = 1.0 if ccw else -1.0
     rings = []
     ring = polygon.exterior
     if signed_area(ring.coords) / s >= 0.0:

pygeoif/functions.py

@@ -17,13 +17,18 @@
 #   Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 #
 """Functions for geometries."""
+import math
 from itertools import groupby
+from itertools import zip_longest
 from typing import Iterable
 from typing import List
 from typing import Tuple
+from typing import Union
 from typing import cast
 
+from pygeoif.types import CoordinatesType
 from pygeoif.types import LineType
+from pygeoif.types import MultiCoordinatesType
 from pygeoif.types import Point2D
 
 
@@ -133,4 +138,28 @@ def dedupe(coords: LineType) -> LineType:
     return tuple(coord for coord, _count in groupby(coords))
 
 
+def compare_coordinates(
+    coords: Union[float, CoordinatesType, MultiCoordinatesType],
+    other: Union[float, CoordinatesType, MultiCoordinatesType],
+) -> bool:
+    """Compare two coordinate sequences."""
+    try:
+        return all(
+            compare_coordinates(c, o)
+            for c, o in zip_longest(
+                coords,  # type: ignore [arg-type]
+                other,  # type: ignore [arg-type]
+                fillvalue=math.nan,
+            )
+        )
+    except TypeError:
+        try:
+            # assert only used to make mypy happy
+            assert isinstance(coords, float)  # noqa: S101
+            assert isinstance(other, float)  # noqa: S101
+            return math.isclose(coords, other)
+        except (TypeError, AssertionError):
+            return coords == other
+
+
 __all__ = ["centroid", "convex_hull", "dedupe", "signed_area"]