Skip to content

chore: modernize packaging, CI, and tooling for Python 3.9+#755

Open
MIDHUNGRAJ wants to merge 3 commits intoXPixelGroup:masterfrom
MIDHUNGRAJ:master
Open

chore: modernize packaging, CI, and tooling for Python 3.9+#755
MIDHUNGRAJ wants to merge 3 commits intoXPixelGroup:masterfrom
MIDHUNGRAJ:master

Conversation

@MIDHUNGRAJ
Copy link

The project hasn't been updated since 2022. This PR modernizes the
infrastructure without touching any model or library code.

Changes:

  • Replace setup.py → pyproject.toml (PEP 517, Python 3.9–3.12)
  • requirements.txt: remove unused future/addict, replace tb-nightly
    with tensorboard>=2.12, update all version pins
  • GitHub Actions: checkout@v4, Python 3.11, OIDC PyPI publishing
  • Replace flake8+yapf with ruff (faster, modern linter+formatter)
  • Add CI matrix tests (Python 3.9/3.10/3.11)
  • Update pre-commit hooks to latest versions
  • ReadTheDocs: ubuntu-24.04, Python 3.11
  • Fix E721 bug in hifacegan_model.py (type() == isinstance)
  • Remove dead badges (LGTM shutdown, glitch.me)
- Fix pyproject.toml: remove invalid flake8-only W503/W504 rule codes from ruff config - Run ruff check --fix: auto-fix 87 import/style issues - Fix E721 in hifacegan_model.py: type() == list -> isinstance(pred, list) - Run ruff format: reformat 101 files to match ruff style (single quotes, spacing) - All lint checks now pass cleanly
- pylint.yml: skip markdown in codespell, add ignore-words-list for existing upstream typos (propgation, simuator, ramdom, uper) - ci.yml: install pytest explicitly + use --no-build-isolation to avoid cython build requirement; ignore test_data (needs local image files) and test_losses (needs CUDA) in CI matrix - All checks verified passing locally: ruff, codespell, pytest (2 passed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant