Repository: claritychallenge/clarity (language: python)

Issue:
Switch to setuptools_scm and pyproject.toml
Rather than using [versioneer](https://github.com/python-versioneer/python-versioneer) to determine versions dynamically the same functionality is provided by [setuptools_scm](https://github.com/pypa/setuptools_scm/) and is cleaner/easier to install maintain as configuration is via the projects `pyproject.toml`.

At the same time switching over to [`pyproject.toml`](https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html) should be undertaken as it is becoming the _de-facto_ method of configuring Python Packages.

* [x] Replace `versioneer` with `setuptools_scm`.
  * [x] `dynamic = ["version"]`
  * [x] `requires = ["setuptools >= 45", "setuptools_scm[toml]>=6.2", "wheel"]`
* [x] Update `docs/conf.py` to use `setuptools_scm`
* [x] Copy `setup.cfg` settings to `pyproject.toml`.
* [x] Add `check_toml` hook to `.pre-commit-config.yaml`.
* [x] Add [`Flake8-pyproject`](https://pypi.org/project/Flake8-pyproject/) as...
  * [x] `additional_dependencies` for `flake8` pre-commit hook.
  * [x] `project.optional.dependencies > dev` in `pyproject.toml`.
* [x] Configure [`ruff`](https://github.com/charliermarsh/ruff) in `pyproject.toml`.
* [x] Remove `setup.cfg` and `setup.py`.
Jpb/204 de duplicate files
These have now been dealt with by variously,
I) deleting copies of files and making code edits so the remaining copy of the file is referenced
ii) editing a copy of the duplicated file to introduce an explicit difference (e.g. in a header comment) - this is useful where there is good reason to have a duplicate copy, e.g. in recipes that are best maintained independently of each other.



Discussion / hints:

Fix patch:
diff --git a/.gitignore b/.gitignore
index f7a061526..ea549f12d 100644
--- a/.gitignore
+++ b/.gitignore
@@ -61,6 +61,9 @@ venv.bak/
 docs/clarity.*.rst
 docs/recipes.*.rst
 
+# setuptools_scm version
+clarity/_version.py
+
 # Emacs
 *~
 
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 60cd65df4..f06adb722 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -17,6 +17,7 @@ repos:
       - id: debug-statements
       - id: check-byte-order-marker
       # - id: check-yaml
+      - id: check-toml
 
   - repo: https://github.com/asottile/pyupgrade
     rev: v3.3.1
@@ -40,7 +41,7 @@ repos:
     rev: 6.0.0
     hooks:
       - id: flake8
-        additional_dependencies: [flake8-print]
+        additional_dependencies: [flake8-print, Flake8-pyproject]
         types: [python]
 
   - repo: https://github.com/pycqa/isort
@@ -93,6 +94,7 @@ repos:
         language: system
         files: \.py$
         exclude: clarity/_version.py # 3rd party code that doesn't conform
+
 ci:
   autofix_prs: true
   autofix_commit_msg: "[pre-commit.ci] Fixing issues with pre-commit"
diff --git a/.pylintrc b/.pylintrc
index 9a531134d..5e93ce3b6 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -21,10 +21,11 @@ fail-under=10.0
 
 # Files or directories to be skipped. They should be base names, not paths.
 ignore=CVS
+       conf.py
 
 # Add files or directories matching the regex patterns to the ignore-list. The
 # regex matches against paths and can be in Posix or Windows format.
-ignore-paths=
+ignore-paths=clarity/_version.py
 
 # Files or directories matching the regex patterns are skipped. The regex
 # matches against base names, not paths.
@@ -45,7 +46,7 @@ limit-inference-results=100
 
 # List of plugins (as comma separated values of python module names) to load,
 # usually to register additional checkers.
-load-plugins=pylint_pytest
+load-plugins=
 
 # Pickle collected data for later comparisons.
 persistent=yes
diff --git a/clarity/__init__.py b/clarity/__init__.py
index 4620d0c9f..9f9bb6c54 100644
--- a/clarity/__init__.py
+++ b/clarity/__init__.py
@@ -1,4 +1,8 @@
 """pyClarity"""
-from . import _version
+from importlib.metadata import PackageNotFoundError, version
 
-__version__ = _version.get_versions()["version"]
+try:
+    __version__ = version("package-name")
+except PackageNotFoundError:
+    # package is not installed
+    pass
diff --git a/clarity/_version.py b/clarity/_version.py
index 0f8ad9db4..ac74faffb 100644
--- a/clarity/_version.py
+++ b/clarity/_version.py
@@ -1,696 +1,4 @@
-# This file helps to compute a version number in source trees obtained from
-# git-archive tarball (such as those provided by githubs download-from-tag
-# feature). Distribution tarballs (built by setup.py sdist) and build
-# directories (produced by setup.py build) will contain a much shorter file
-# that just contains the computed version number.
-
-# This file is released into the public domain.
-# Generated by versioneer-0.26
-# https://github.com/python-versioneer/python-versioneer
-
-# pylint: disable=W0201  # allow "attribute defined outside __init__"
-
-"""Git implementation of _version.py."""
-
-import errno
-import functools
-import os
-import re
-import subprocess
-import sys
-from typing import Callable, Dict
-
-
-def get_keywords():
-    """Get the keywords needed to look up the version information."""
-    # these strings will be replaced by git during git-archive.
-    # setup.py/versioneer.py will grep for the variable names, so they must
-    # each be defined on a line of their own. _version.py will just call
-    # get_keywords().
-    git_refnames = "$Format:%d$"
-    git_full = "$Format:%H$"
-    git_date = "$Format:%ci$"
-    keywords = {"refnames": git_refnames, "full": git_full, "date": git_date}
-    return keywords
-
-
-class VersioneerConfig:
-    """Container for Versioneer configuration parameters."""
-
-
-def get_config():
-    """Create, populate and return the VersioneerConfig() object."""
-    # these strings are filled in when 'setup.py versioneer' creates
-    # _version.py
-    cfg = VersioneerConfig()
-    cfg.VCS = "git"
-    cfg.style = "pep440"
-    cfg.tag_prefix = "v"
-    cfg.parentdir_prefix = ""
-    cfg.versionfile_source = "clarity/_version.py"
-    cfg.verbose = False
-    return cfg
-
-
-class NotThisMethod(Exception):
-    """Exception raised if a method is not valid for the current scenario."""
-
-
-LONG_VERSION_PY: Dict[str, str] = {}
-HANDLERS: Dict[str, Dict[str, Callable]] = {}
-
-
-def register_vcs_handler(vcs, method):  # decorator
-    """Create decorator to mark a method as the handler of a VCS."""
-
-    def decorate(f):
-        """Store f in HANDLERS[vcs][method]."""
-        if vcs not in HANDLERS:
-            HANDLERS[vcs] = {}
-        HANDLERS[vcs][method] = f
-        return f
-
-    return decorate
-
-
-def run_command(commands, args, cwd=None, verbose=False, hide_stderr=False, env=None):
-    """Call the given command(s)."""
-    assert isinstance(commands, list)
-    process = None
-
-    popen_kwargs = {}
-    if sys.platform == "win32":
-        # This hides the console window if pythonw.exe is used
-        startupinfo = subprocess.STARTUPINFO()
-        startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
-        popen_kwargs["startupinfo"] = startupinfo
-
-    for command in commands:
-        dispcmd = None
-        try:
-            dispcmd = str([command] + args)
-            # remember shell=False, so use git.cmd on windows, not just git
-            process = subprocess.Popen(  # pylint: disable=consider-using-with
-                [command] + args,
-                cwd=cwd,
-                env=env,
-                stdout=subprocess.PIPE,
-                stderr=(subprocess.PIPE if hide_stderr else None),
-                **popen_kwargs,
-            )
-            break
-        except OSError:
-            e = sys.exc_info()[1]
-            if e.errno == errno.ENOENT:
-                continue
-            if verbose:
-                print("unable to run %s" % dispcmd)
-                print(e)
-            return None, None
-    else:
-        if verbose:
-            print(f"unable to find command, tried {commands}")
-        return None, None
-    stdout = process.communicate()[0].strip().decode()
-    if process.returncode != 0:
-        if verbose:
-            print("unable to run %s (error)" % dispcmd)
-            print("stdout was %s" % stdout)
-        return None, process.returncode
-    return stdout, process.returncode
-
-
-def versions_from_parentdir(parentdir_prefix, root, verbose):
-    """Try to determine the version from the parent directory name.
-
-    Source tarballs conventionally unpack into a directory that includes both
-    the project name and a version string. We will also support searching up
-    two directory levels for an appropriately named parent directory
-    """
-    rootdirs = []
-
-    for _ in range(3):
-        dirname = os.path.basename(root)
-        if dirname.startswith(parentdir_prefix):
-            return {
-                "version": dirname[len(parentdir_prefix) :],
-                "full-revisionid": None,
-                "dirty": False,
-                "error": None,
-                "date": None,
-            }
-        rootdirs.append(root)
-        root = os.path.dirname(root)  # up a level
-
-    if verbose:
-        print(
-            "Tried directories %s but none started with prefix %s"
-            % (str(rootdirs), parentdir_prefix)
-        )
-    raise NotThisMethod("rootdir doesn't start with parentdir_prefix")
-
-
-@register_vcs_handler("git", "get_keywords")
-def git_get_keywords(versionfile_abs):
-    """Extract version information from the given file."""
-    # the code embedded in _version.py can just fetch the value of these
-    # keywords. When used from setup.py, we don't want to import _version.py,
-    # so we do it with a regexp instead. This function is not used from
-    # _version.py.
-    keywords = {}
-    try:
-        with open(versionfile_abs, encoding="utf-8") as fobj:
-            for line in fobj:
-                if line.strip().startswith("git_refnames ="):
-                    mo = re.search(r'=\s*"(.*)"', line)
-                    if mo:
-                        keywords["refnames"] = mo.group(1)
-                if line.strip().startswith("git_full ="):
-                    mo = re.search(r'=\s*"(.*)"', line)
-                    if mo:
-                        keywords["full"] = mo.group(1)
-                if line.strip().startswith("git_date ="):
-                    mo = re.search(r'=\s*"(.*)"', line)
-                    if mo:
-                        keywords["date"] = mo.group(1)
-    except OSError:
-        pass
-    return keywords
-
-
-@register_vcs_handler("git", "keywords")
-def git_versions_from_keywords(keywords, tag_prefix, verbose):
-    """Get version information from git keywords."""
-    if "refnames" not in keywords:
-        raise NotThisMethod("Short version file found")
-    date = keywords.get("date")
-    if date is not None:
-        # Use only the last line.  Previous lines may contain GPG signature
-        # information.
-        date = date.splitlines()[-1]
-
-        # git-2.2.0 added "%cI", which expands to an ISO-8601 -compliant
-        # datestamp. However we prefer "%ci" (which expands to an "ISO-8601
-        # -like" string, which we must then edit to make compliant), because
-        # it's been around since git-1.5.3, and it's too difficult to
-        # discover which version we're using, or to work around using an
-        # older one.
-        date = date.strip().replace(" ", "T", 1).replace(" ", "", 1)
-    refnames = keywords["refnames"].strip()
-    if refnames.startswith("$Format"):
-        if verbose:
-            print("keywords are unexpanded, not using")
-        raise NotThisMethod("unexpanded keywords, not a git-archive tarball")
-    refs = {r.strip() for r in refnames.strip("()").split(",")}
-    # starting in git-1.8.3, tags are listed as "tag: foo-1.0" instead of
-    # just "foo-1.0". If we see a "tag: " prefix, prefer those.
-    TAG = "tag: "
-    tags = {r[len(TAG) :] for r in refs if r.startswith(TAG)}
-    if not tags:
-        # Either we're using git < 1.8.3, or there really are no tags. We use
-        # a heuristic: assume all version tags have a digit. The old git %d
-        # expansion behaves like git log --decorate=short and strips out the
-        # refs/heads/ and refs/tags/ prefixes that would let us distinguish
-        # between branches and tags. By ignoring refnames without digits, we
-        # filter out many common branch names like "release" and
-        # "stabilization", as well as "HEAD" and "master".
-        tags = {r for r in refs if re.search(r"\d", r)}
-        if verbose:
-            print("discarding '%s', no digits" % ",".join(refs - tags))
-    if verbose:
-        print("likely tags: %s" % ",".join(sorted(tags)))
-    for ref in sorted(tags):
-        # sorting will prefer e.g. "2.0" over "2.0rc1"
-        if ref.startswith(tag_prefix):
-            r = ref[len(tag_prefix) :]
-            # Filter out refs that exactly match prefix or that don't start
-            # with a number once the prefix is stripped (mostly a concern
-            # when prefix is '')
-            if not re.match(r"\d", r):
-                continue
-            if verbose:
-                print("picking %s" % r)
-            return {
-                "version": r,
-                "full-revisionid": keywords["full"].strip(),
-                "dirty": False,
-                "error": None,
-                "date": date,
-            }
-    # no suitable tags, so version is "0+unknown", but full hex is still there
-    if verbose:
-        print("no suitable tags, using unknown + full revision id")
-    return {
-        "version": "0+unknown",
-        "full-revisionid": keywords["full"].strip(),
-        "dirty": False,
-        "error": "no suitable tags",
-        "date": None,
-    }
-
-
-@register_vcs_handler("git", "pieces_from_vcs")
-def git_pieces_from_vcs(tag_prefix, root, verbose, runner=run_command):
-    """Get version from 'git describe' in the root of the source tree.
-
-    This only gets called if the git-archive 'subst' keywords were *not*
-    expanded, and _version.py hasn't already been rewritten with a short
-    version string, meaning we're inside a checked out source tree.
-    """
-    GITS = ["git"]
-    if sys.platform == "win32":
-        GITS = ["git.cmd", "git.exe"]
-
-    # GIT_DIR can interfere with correct operation of Versioneer.
-    # It may be intended to be passed to the Versioneer-versioned project,
-    # but that should not change where we get our version from.
-    env = os.environ.copy()
-    env.pop("GIT_DIR", None)
-    runner = functools.partial(runner, env=env)
-
-    _, rc = runner(GITS, ["rev-parse", "--git-dir"], cwd=root, hide_stderr=not verbose)
-    if rc != 0:
-        if verbose:
-            print("Directory %s not under git control" % root)
-        raise NotThisMethod("'git rev-parse --git-dir' returned error")
-
-    # if there is a tag matching tag_prefix, this yields TAG-NUM-gHEX[-dirty]
-    # if there isn't one, this yields HEX[-dirty] (no NUM)
-    describe_out, rc = runner(
-        GITS,
-        [
-            "describe",
-            "--tags",
-            "--dirty",
-            "--always",
-            "--long",
-            "--match",
-            f"{tag_prefix}[[:digit:]]*",
-        ],
-        cwd=root,
-    )
-    # --long was added in git-1.5.5
-    if describe_out is None:
-        raise NotThisMethod("'git describe' failed")
-    describe_out = describe_out.strip()
-    full_out, rc = runner(GITS, ["rev-parse", "HEAD"], cwd=root)
-    if full_out is None:
-        raise NotThisMethod("'git rev-parse' failed")
-    full_out = full_out.strip()
-
-    pieces = {}
-    pieces["long"] = full_out
-    pieces["short"] = full_out[:7]  # maybe improved later
-    pieces["error"] = None
-
-    branch_name, rc = runner(GITS, ["rev-parse", "--abbrev-ref", "HEAD"], cwd=root)
-    # --abbrev-ref was added in git-1.6.3
-    if rc != 0 or branch_name is None:
-        raise NotThisMethod("'git rev-parse --abbrev-ref' returned error")
-    branch_name = branch_name.strip()
-
-    if branch_name == "HEAD":
-        # If we aren't exactly on a branch, pick a branch which represents
-        # the current commit. If all else fails, we are on a branchless
-        # commit.
-        branches, rc = runner(GITS, ["branch", "--contains"], cwd=root)
-        # --contains was added in git-1.5.4
-        if rc != 0 or branches is None:
-            raise NotThisMethod("'git branch --contains' returned error")
-        branches = branches.split("\n")
-
-        # Remove the first line if we're running detached
-        if "(" in branches[0]:
-            branches.pop(0)
-
-        # Strip off the leading "* " from the list of branches.
-        branches = [branch[2:] for branch in branches]
-        if "master" in branches:
-            branch_name = "master"
-        elif not branches:
-            branch_name = None
-        else:
-            # Pick the first branch that is returned. Good or bad.
-            branch_name = branches[0]
-
-    pieces["branch"] = branch_name
-
-    # parse describe_out. It will be like TAG-NUM-gHEX[-dirty] or HEX[-dirty]
-    # TAG might have hyphens.
-    git_describe = describe_out
-
-    # look for -dirty suffix
-    dirty = git_describe.endswith("-dirty")
-    pieces["dirty"] = dirty
-    if dirty:
-        git_describe = git_describe[: git_describe.rindex("-dirty")]
-
-    # now we have TAG-NUM-gHEX or HEX
-
-    if "-" in git_describe:
-        # TAG-NUM-gHEX
-        mo = re.search(r"^(.+)-(\d+)-g([0-9a-f]+)$", git_describe)
-        if not mo:
-            # unparsable. Maybe git-describe is misbehaving?
-            pieces["error"] = "unable to parse git-describe output: '%s'" % describe_out
-            return pieces
-
-        # tag
-        full_tag = mo.group(1)
-        if not full_tag.startswith(tag_prefix):
-            if verbose:
-                fmt = "tag '%s' doesn't start with prefix '%s'"
-                print(fmt % (full_tag, tag_prefix))
-            pieces["error"] = "tag '{}' doesn't start with prefix '{}'".format(
-                full_tag,
-                tag_prefix,
-            )
-            return pieces
-        pieces["closest-tag"] = full_tag[len(tag_prefix) :]
-
-        # distance: number of commits since tag
-        pieces["distance"] = int(mo.group(2))
-
-        # commit: short hex revision ID
-        pieces["short"] = mo.group(3)
-
-    else:
-        # HEX: no tags
-        pieces["closest-tag"] = None
-        out, rc = runner(GITS, ["rev-list", "HEAD", "--left-right"], cwd=root)
-        pieces["distance"] = len(out.split())  # total number of commits
-
-    # commit date: see ISO-8601 comment in git_versions_from_keywords()
-    date = runner(GITS, ["show", "-s", "--format=%ci", "HEAD"], cwd=root)[0].strip()
-    # Use only the last line.  Previous lines may contain GPG signature
-    # information.
-    date = date.splitlines()[-1]
-    pieces["date"] = date.strip().replace(" ", "T", 1).replace(" ", "", 1)
-
-    return pieces
-
-
-def plus_or_dot(pieces):
-    """Return a + if we don't already have one, else return a ."""
-    if "+" in pieces.get("closest-tag", ""):
-        return "."
-    return "+"
-
-
-def render_pep440(pieces):
-    """Build up version string, with post-release "local version identifier".
-
-    Our goal: TAG[+DISTANCE.gHEX[.dirty]] . Note that if you
-    get a tagged build and then dirty it, you'll get TAG+0.gHEX.dirty
-
-    Exceptions:
-    1: no tags. git_describe was just HEX. 0+untagged.DISTANCE.gHEX[.dirty]
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"] or pieces["dirty"]:
-            rendered += plus_or_dot(pieces)
-            rendered += "%d.g%s" % (pieces["distance"], pieces["short"])
-            if pieces["dirty"]:
-                rendered += ".dirty"
-    else:
-        # exception #1
-        rendered = "0+untagged.%d.g%s" % (pieces["distance"], pieces["short"])
-        if pieces["dirty"]:
-            rendered += ".dirty"
-    return rendered
-
-
-def render_pep440_branch(pieces):
-    """TAG[[.dev0]+DISTANCE.gHEX[.dirty]] .
-
-    The ".dev0" means not master branch. Note that .dev0 sorts backwards
-    (a feature branch will appear "older" than the master branch).
-
-    Exceptions:
-    1: no tags. 0[.dev0]+untagged.DISTANCE.gHEX[.dirty]
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"] or pieces["dirty"]:
-            if pieces["branch"] != "master":
-                rendered += ".dev0"
-            rendered += plus_or_dot(pieces)
-            rendered += "%d.g%s" % (pieces["distance"], pieces["short"])
-            if pieces["dirty"]:
-                rendered += ".dirty"
-    else:
-        # exception #1
-        rendered = "0"
-        if pieces["branch"] != "master":
-            rendered += ".dev0"
-        rendered += "+untagged.%d.g%s" % (pieces["distance"], pieces["short"])
-        if pieces["dirty"]:
-            rendered += ".dirty"
-    return rendered
-
-
-def pep440_split_post(ver):
-    """Split pep440 version string at the post-release segment.
-
-    Returns the release segments before the post-release and the
-    post-release version number (or -1 if no post-release segment is present).
-    """
-    vc = str.split(ver, ".post")
-    return vc[0], int(vc[1] or 0) if len(vc) == 2 else None
-
-
-def render_pep440_pre(pieces):
-    """TAG[.postN.devDISTANCE] -- No -dirty.
-
-    Exceptions:
-    1: no tags. 0.post0.devDISTANCE
-    """
-    if pieces["closest-tag"]:
-        if pieces["distance"]:
-            # update the post release segment
-            tag_version, post_version = pep440_split_post(pieces["closest-tag"])
-            rendered = tag_version
-            if post_version is not None:
-                rendered += ".post%d.dev%d" % (post_version + 1, pieces["distance"])
-            else:
-                rendered += ".post0.dev%d" % (pieces["distance"])
-        else:
-            # no commits, use the tag as the version
-            rendered = pieces["closest-tag"]
-    else:
-        # exception #1
-        rendered = "0.post0.dev%d" % pieces["distance"]
-    return rendered
-
-
-def render_pep440_post(pieces):
-    """TAG[.postDISTANCE[.dev0]+gHEX] .
-
-    The ".dev0" means dirty. Note that .dev0 sorts backwards
-    (a dirty tree will appear "older" than the corresponding clean one),
-    but you shouldn't be releasing software with -dirty anyways.
-
-    Exceptions:
-    1: no tags. 0.postDISTANCE[.dev0]
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"] or pieces["dirty"]:
-            rendered += ".post%d" % pieces["distance"]
-            if pieces["dirty"]:
-                rendered += ".dev0"
-            rendered += plus_or_dot(pieces)
-            rendered += "g%s" % pieces["short"]
-    else:
-        # exception #1
-        rendered = "0.post%d" % pieces["distance"]
-        if pieces["dirty"]:
-            rendered += ".dev0"
-        rendered += "+g%s" % pieces["short"]
-    return rendered
-
-
-def render_pep440_post_branch(pieces):
-    """TAG[.postDISTANCE[.dev0]+gHEX[.dirty]] .
-
-    The ".dev0" means not master branch.
-
-    Exceptions:
-    1: no tags. 0.postDISTANCE[.dev0]+gHEX[.dirty]
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"] or pieces["dirty"]:
-            rendered += ".post%d" % pieces["distance"]
-            if pieces["branch"] != "master":
-                rendered += ".dev0"
-            rendered += plus_or_dot(pieces)
-            rendered += "g%s" % pieces["short"]
-            if pieces["dirty"]:
-                rendered += ".dirty"
-    else:
-        # exception #1
-        rendered = "0.post%d" % pieces["distance"]
-        if pieces["branch"] != "master":
-            rendered += ".dev0"
-        rendered += "+g%s" % pieces["short"]
-        if pieces["dirty"]:
-            rendered += ".dirty"
-    return rendered
-
-
-def render_pep440_old(pieces):
-    """TAG[.postDISTANCE[.dev0]] .
-
-    The ".dev0" means dirty.
-
-    Exceptions:
-    1: no tags. 0.postDISTANCE[.dev0]
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"] or pieces["dirty"]:
-            rendered += ".post%d" % pieces["distance"]
-            if pieces["dirty"]:
-                rendered += ".dev0"
-    else:
-        # exception #1
-        rendered = "0.post%d" % pieces["distance"]
-        if pieces["dirty"]:
-            rendered += ".dev0"
-    return rendered
-
-
-def render_git_describe(pieces):
-    """TAG[-DISTANCE-gHEX][-dirty].
-
-    Like 'git describe --tags --dirty --always'.
-
-    Exceptions:
-    1: no tags. HEX[-dirty]  (note: no 'g' prefix)
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        if pieces["distance"]:
-            rendered += "-%d-g%s" % (pieces["distance"], pieces["short"])
-    else:
-        # exception #1
-        rendered = pieces["short"]
-    if pieces["dirty"]:
-        rendered += "-dirty"
-    return rendered
-
-
-def render_git_describe_long(pieces):
-    """TAG-DISTANCE-gHEX[-dirty].
-
-    Like 'git describe --tags --dirty --always -long'.
-    The distance/hash is unconditional.
-
-    Exceptions:
-    1: no tags. HEX[-dirty]  (note: no 'g' prefix)
-    """
-    if pieces["closest-tag"]:
-        rendered = pieces["closest-tag"]
-        rendered += "-%d-g%s" % (pieces["distance"], pieces["short"])
-    else:
-        # exception #1
-        rendered = pieces["short"]
-    if pieces["dirty"]:
-        rendered += "-dirty"
-    return rendered
-
-
-def render(pieces, style):
-    """Render the given version pieces into the requested style."""
-    if pieces["error"]:
-        return {
-            "version": "unknown",
-            "full-revisionid": pieces.get("long"),
-            "dirty": None,
-            "error": pieces["error"],
-            "date": None,
-        }
-
-    if not style or style == "default":
-        style = "pep440"  # the default
-
-    if style == "pep440":
-        rendered = render_pep440(pieces)
-    elif style == "pep440-branch":
-        rendered = render_pep440_branch(pieces)
-    elif style == "pep440-pre":
-        rendered = render_pep440_pre(pieces)
-    elif style == "pep440-post":
-        rendered = render_pep440_post(pieces)
-    elif style == "pep440-post-branch":
-        rendered = render_pep440_post_branch(pieces)
-    elif style == "pep440-old":
-        rendered = render_pep440_old(pieces)
-    elif style == "git-describe":
-        rendered = render_git_describe(pieces)
-    elif style == "git-describe-long":
-        rendered = render_git_describe_long(pieces)
-    else:
-        raise ValueError("unknown style '%s'" % style)
-
-    return {
-        "version": rendered,
-        "full-revisionid": pieces["long"],
-        "dirty": pieces["dirty"],
-        "error": None,
-        "date": pieces.get("date"),
-    }
-
-
-def get_versions():
-    """Get version information or return default if unable to do so."""
-    # I am in _version.py, which lives at ROOT/VERSIONFILE_SOURCE. If we have
-    # __file__, we can work backwards from there to the root. Some
-    # py2exe/bbfreeze/non-CPython implementations don't do __file__, in which
-    # case we can only use expanded keywords.
-
-    cfg = get_config()
-    verbose = cfg.verbose
-
-    try:
-        return git_versions_from_keywords(get_keywords(), cfg.tag_prefix, verbose)
-    except NotThisMethod:
-        pass
-
-    try:
-        root = os.path.realpath(__file__)
-        # versionfile_source is the relative path from the top of the source
-        # tree (where the .git directory might live) to this file. Invert
-        # this to find the root from __file__.
-        for _ in cfg.versionfile_source.split("/"):
-            root = os.path.dirname(root)
-    except NameError:
-        return {
-            "version": "0+unknown",
-            "full-revisionid": None,
-            "dirty": None,
-            "error": "unable to find root of source tree",
-            "date": None,
-        }
-
-    try:
-        pieces = git_pieces_from_vcs(cfg.tag_prefix, root, verbose)
-        return render(pieces, cfg.style)
-    except NotThisMethod:
-        pass
-
-    try:
-        if cfg.parentdir_prefix:
-            return versions_from_parentdir(cfg.parentdir_prefix, root, verbose)
-    except NotThisMethod:
-        pass
-
-    return {
-        "version": "0+unknown",
-        "full-revisionid": None,
-        "dirty": None,
-        "error": "unable to compute version",
-        "date": None,
-    }
+# file generated by setuptools_scm
+# don't change, don't track in version control
+__version__ = version = "0.2.2.dev508+g2611c0d.d20230327"
+__version_tuple__ = version_tuple = (0, 2, 2, "dev508", "g2611c0d.d20230327")
diff --git a/docs/conf.py b/docs/conf.py
index 27a645b87..e70c36fd7 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -1,13 +1,13 @@
+"""Sphinx configuration"""
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
 # documentation root, use os.path.abspath to make it absolute, like shown here.
 #
 import os
 import sys
+from importlib.metadata import version
 from typing import Dict
 
-import clarity
-
 # -*- coding: utf-8 -*-
 #
 # Configuration file for the Sphinx documentation builder.
@@ -38,14 +38,13 @@
 
 # -- Project information -----------------------------------------------------
 
-project = "pyclarity"
-copyright = "2022, pyclarity authors"  # pylint: disable=redefined-builtin
-author = "pyclarity authors"
+PROJECT = "pyclarity"
+COPYRIGHT = "2022, pyclarity authors"
+AUTHOR = "pyclarity authors"
 
-# The short X.Y version
-version = ""
-# The full version, including alpha/beta/rc tags
-release = clarity.__version__
+RELEASE = version("pyclarity")
+# for example take major/minor
+VERSION = ".".join(RELEASE.split(".")[:2])  # type: ignore
 
 
 # -- General configuration ---------------------------------------------------
@@ -85,14 +84,14 @@
 }
 
 # The master toctree document.
-master_doc = "index"
+MASTER_DOC = "index"
 
 # The language for content autogenerated by Sphinx. Refer to documentation
 # for a list of supported languages.
 #
 # This is also used if you do content translation via gettext catalogs.
 # Usually you set "language" from the command line for these cases.
-language = "en"
+LANGUAGE = "en"
 
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.
@@ -100,7 +99,7 @@
 exclude_patterns = ["_build", "Thumbs.db", ".DS_Store"]
 
 # The name of the Pygments (syntax highlighting) style to use.
-pygments_style = None
+PYGMENTS_STYLE = None
 
 
 # -- Options for HTML output -------------------------------------------------
@@ -111,7 +110,7 @@
 # html_theme = 'sphinx_rtd_theme'
 # html_theme = "haiku"
 # html_theme = "renku"
-html_theme = "pydata_sphinx_theme"
+HTML_THEME = "pydata_sphinx_theme"
 
 # Theme options are theme-specific and customize the look and feel of a theme
 # further.  For a list of options available for each theme, see the
@@ -123,13 +122,12 @@
 #         {
 #             "rel": "icon",
 #             "sizes": "156x20",
-#             "href": "https://github.com/Mesnage-Org/clarity/actions/workflows/\
-#                           ci-tests.yml/badge.svg",
+#             "href": "",
 #         },
 #         {
 #             "rel": "icon",
 #             "sizes": "115x20",
-#             "href": "https://codecov.io/gh/Mesnage-Org/clarity",
+#             "href": "",
 #         },
 #     ],
 # }
@@ -153,7 +151,7 @@
 # -- Options for HTMLHelp output ---------------------------------------------
 
 # Output file base name for HTML help builder.
-htmlhelp_basename = "claritydoc"
+HTMLHELP_BASENAME = "claritydoc"
 
 
 # -- Options for LaTeX output ------------------------------------------------
@@ -177,7 +175,7 @@
 # (source start file, target name, title,
 #  author, documentclass [howto, manual, or own class]).
 latex_documents = [
-    (master_doc, "clarity.tex", "clarity Documentation", [author], "manual"),
+    (MASTER_DOC, "clarity.tex", "clarity Documentation", [AUTHOR], "manual"),
 ]
 
 
@@ -185,7 +183,7 @@
 
 # One entry per manual page. List of tuples
 # (source start file, name, description, authors, manual section).
-man_pages = [(master_doc, "clarity", "Clarity Documentation", [author], 1)]
+man_pages = [(MASTER_DOC, "clarity", "Clarity Documentation", [AUTHOR], 1)]
 
 
 # -- Options for Texinfo output ----------------------------------------------
@@ -194,14 +192,14 @@
 # (source start file, target name, title, author,
 #  dir menu entry, description, category)
 texinfo_documents = [
-    (master_doc, "Clarity", "Clarity Documentation", author, "Clarity", "", ""),
+    (MASTER_DOC, "Clarity", "Clarity Documentation", AUTHOR, "Clarity", "", ""),
 ]
 
 
 # -- Options for Epub output -------------------------------------------------
 
 # Bibliographic Dublin Core info.
-epub_title = project
+EPUB_TITLE = PROJECT
 
 # The unique identifier of the text. This can be a ISBN number
 # or the project homepage.
diff --git a/pyproject.toml b/pyproject.toml
index 2862389b8..3571d8f95 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,11 +1,133 @@
 [build-system]
 requires = [
-  "setuptools >= 40.9.0",
+  "setuptools >= 45",
+  "setuptools_scm[toml]>=6.2",
   "wheel",
-  "versioneer"
   ]
 build-backend = "setuptools.build_meta"
 
+[project]
+name = "pyclarity"
+description = "Tools for the Clarity Challenge"
+readme = "README.md"
+license = {text = "MIT"}
+dynamic = ["version"]
+authors = [
+  {name = "The PyClarity Team", email = "claritychallengecontact@gmail.com"},
+]
+classifiers = [
+  "Intended Audience :: Developers",
+  "Intended Audience :: Education",
+  "Intended Audience :: Science/Research",
+  "License :: OSI Approved :: MIT License",
+  "Operating System :: OS Independent",
+  "Programming Language :: Python :: 3",
+  "Natural Language :: English",
+]
+keywords = [
+  "hearing",
+  "signal processing",
+  "clarity challenge"
+]
+requires-python = ">=3.8"
+dependencies = [
+  "audioread>=2.1.9",
+  "gdown",
+  "hydra-core==1.1.1",
+  "hydra-submitit-launcher>=1.1.6",
+  "importlib-metadata",
+  "librosa>=0.8.1",
+  "matplotlib",
+  "numpy>=1.21.6",
+  "omegaconf>=2.1.1",
+  "pandas>=1.3.5",
+  "pyloudnorm>=0.1.0",
+  "pystoi",
+  "resampy",
+  "scikit-learn>=1.0.2",
+  "scipy>=1.7.3",
+  "SoundFile>=0.10.3.post1",
+  "torch<2.0",
+  "torchaudio",
+  "tqdm>=4.62.3",
+  "typing_extensions",
+]
+
+[project.optional-dependencies]
+tests = [
+  "coverage",
+  "isort",
+  "flake8",
+  "flake8-print",
+  "Flake8-pyproject",
+  "mypy",
+  "pre-commit",
+  "py",
+  "py-cpuinfo",
+  "pytest",
+  "pytest-cov",
+  "pytest-mpl",
+  "pytest-regtest",
+  "pytest-xdist",
+  "yamllint",
+]
+docs =[
+  "sphinx",
+  "myst_parser",
+  "pydata_sphinx_theme",
+  "sphinx_markdown_tables",
+  "sphinx_rtd_theme",
+  "sphinxcontrib-mermaid",
+  "sphinxcontrib-napoleon",
+]
+dev = [
+  "black",
+  "pre-commit",
+  "pycodestyle",
+  "pylint",
+  "pylint-pytest",
+  "yamllint",
+]
+pypi =[
+  "build",
+  "wheel",
+  "setuptools_scm[toml]"
+]
+
+[project.urls]
+Source = "https://github.com/claritychallenge/clarity"
+Bug_Tracker = "https://github.com/claritychallenge/clarity/issues"
+Documentation = "https://claritychallenge.github.io/clarity_CC_doc"
+
+[tool.setuptools]
+# packages = ["clarity", "recipes"]
+# packages = "find"
+# py-modules = ["clarity", "recipes"]
+include-package-data = true
+#package-dir = ["clarity", "recipes"]
+
+[tool.setuptools.packages.find]
+include = ["clarity*", "recipes*"]
+exclude = ["tests*"]
+namespaces = false
+
+[tool.setuptools.package-data]
+clarity = [
+  "*.json",
+  "*.mat",
+  "*.yaml"
+]
+recipes = [
+  "*.csv",
+  "*.json",
+  "*.mat",
+  "*.yaml"
+]
+
+
+[tool.setuptools_scm]
+write_to = "clarity/_version.py"
+
 [tool.pytest.ini_options]
 minversion = "7.0"
 addopts = "--cov clarity"
@@ -16,6 +138,15 @@ filterwarnings = [
     "ignore::UserWarning"
 ]
 
+[tool.coverage.run]
+source = ["clarity"]
+omit = [
+  "*conftest.py",
+  "*tests*",
+  "**/__init__*",
+  "clarity/_version.py",
+]
+
 [tool.black]
 line-length = 88
 target-version = ['py37']
@@ -30,21 +161,58 @@ exclude = '''
 )
 '''
 
+[flake8]
+ignore =[
+  "E203",
+  "E501",
+  "W503",
+]
+# docstring-convention = ""
+max-line-length = 88
+max-complexity = 18
+select = [
+  "B",
+  "C",
+  "E",
+  "F",
+  "W",
+  "T4",
+  "B9"
+]
+
+[tool.ruff]
+exclude = [
+    ".bzr",
+    ".direnv",
+    ".eggs",
+    ".git",
+    ".hg",
+    ".mypy_cache",
+    ".nox",
+    ".pants.d",
+    ".pytype",
+    ".ruff_cache",
+    ".svn",
+    ".tox",
+    ".venv",
+    "__pypackages__",
+    "_build",
+    "buck-out",
+    "build",
+    "dist",
+    "node_modules",
+    "venv",
+]
+# per-file-ignores = []
+line-length = 88
+target-version = "py37"
+select = ["B", "E", "F", "I001", "W"]
+# Allow autofix for all enabled rules (when `--fix`) is provided.
+fixable = ["A", "B", "C", "D", "E", "F", "R", "S", "W", "U"]
+
 [tool.mypy]
 ignore_missing_imports = true
 exclude = [
     "docs/*",
     "build/*"
-    ]
-
-[tool.coverage.run]
-omit = [
-    "clarity/_version.py"
 ]
-
-
-[tool.ruff]
-line-length = 88
-# pyflakes, pycodestyle, isort
-select = ["F", "E", "W", "B", "I001"]
-ignore = ["B905"]
\ No newline at end of file
diff --git a/setup.cfg b/setup.cfg
deleted file mode 100644
index 991e8d987..000000000
--- a/setup.cfg
+++ /dev/null
@@ -1,139 +0,0 @@
-[metadata]
-name = pyclarity
-description = Tools for the Clarity Challenge
-long_description = file: README.md,
-long_description_content_type = text/markdown
-author = The PyClarity team
-author_email = clarity-group@sheffield.ac.uk
-url = https://github.com/claritychallenge/clarity
-project_urls=
-  Bug Tracker = https://github.com/claritychallenge/clarity/issues
-  Documentation = https://claritychallenge.github.io/clarity_CC_doc
-  Source = https://github.com/claritychallenge/clarity
-license = MIT
-keywords = machine learning, challenges, speech-enhancement, hearing-aids
-classifiers=
-    Intended Audience :: Developers
-    Intended Audience :: Education
-#    Intended Audience :: Hearing Industry
-    Intended Audience :: Science/Research
-    License :: OSI Approved :: MIT License
-    Operating System :: OS Independent
-    Programming Language :: Python :: 3
-    Natural Language :: English
-
-
-[options]
-python_requires = >=3.7
-
-zip_safe = False
-
-include_package_data = True
-
-packages = find:
-
-install_requires =
-  audioread>=2.1.9
-  fastdtw
-  gdown
-  hydra-core==1.1.1
-  hydra-submitit-launcher>=1.1.6
-  importlib-metadata
-  librosa>=0.8.1
-  matplotlib
-  numpy>=1.21.6
-  omegaconf>=2.1.1
-  pandas>=1.3.5
-  pyloudnorm>=0.1.0
-  pytorch-lightning
-  pystoi
-  resampy
-  scikit-learn>=1.0.2
-  scipy>=1.7.3
-  SoundFile>=0.10.3.post1
-  speechbrain
-  torch<2.0
-  torchaudio
-  tqdm>=4.62.3
-  typing_extensions
-
-[options.extras_require]
-tests =
-  isort
-  flake8
-  mypy
-  pre-commit
-  py
-  py-cpuinfo
-  pytest
-  pytest-cov
-  pytest-mpl
-  pytest-regtest
-  yamllint
-
-docs =
-  sphinx
-  myst_parser
-  pydata_sphinx_theme
-  sphinx_markdown_tables
-  sphinx_rtd_theme
-  sphinxcontrib-mermaid
-  sphinxcontrib-napoleon
-
-dev =
-  black
-  coverage
-  flake8
-  flake8-print
-  isort
-  mypy
-  pre-commit
-  pycodestyle
-  pylint
-  pylint-pytest
-  pytest
-  pytest-cov
-  pytest-regtest
-  yamllint
-
-pypi =
-  build
-  versioneer
-  wheel
-
-[options.package_data]
-
-* =
-  *.csv
-  *.json
-  *.mat
-  *.yaml
-
-[flake8]
-ignore = E203, E501, W503
-max-line-length = 88
-max-complexity = 18
-select = B,C,E,F,W,T4,B9
-
-[options.packages.find]
-exclude = tests*
-
-[coverage:run]
-omit =
-  *conftest.py
-  *tests*
-  **/__init__*
-source =
-    clarity
-
-# FIXME : Is an entry_point required (https://setuptools.pypa.io/en/latest/userguide/entry_point.html)
-# [options.entry_points]
-# console_scripts =
-
-[versioneer]
-VCS = git
-style = pep440
-versionfile_source = clarity/_version.py
-versionfile_build = clarity/_version.py
-tag_prefix = v
-parentdir_prefix =
\ No newline at end of file
diff --git a/setup.py b/setup.py
deleted file mode 100644
index 530a0572f..000000000
--- a/setup.py
+++ /dev/null
@@ -1,8 +0,0 @@
-"""Package setup"""
-import versioneer
-from setuptools import setup
-
-setup(
-    version=versioneer.get_version(),  # pylint: disable=no-member
-    cmdclass=versioneer.get_cmdclass(),  # pylint: disable=no-member
-)


Test patch:
diff --git a/.github/workflows/run_tests.yml b/.github/workflows/run_tests.yml
index 8368b673c..b953b8879 100644
--- a/.github/workflows/run_tests.yml
+++ b/.github/workflows/run_tests.yml
@@ -16,7 +16,7 @@ jobs:
     strategy:
       matrix:
         os: ["ubuntu-latest"]
-        python-version: ['3.7', '3.8', '3.9', '3.10']
+        python-version: ['3.8', '3.9', '3.10']
     steps:
       - uses: actions/checkout@v3
       - name: Set up Python
@@ -31,10 +31,7 @@ jobs:
           sudo apt-get install -yq libsndfile1-dev ffmpeg
       - name: Install package, including extras
         run: |
-          pip install -e .
-          pip install -e .[tests]
-          pip install -e .[docs]
-          pip install -e .[dev]
+          pip install -e .[tests,docs,dev]
       - name: CPU information
         run: |
           python -m cpuinfo --json