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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant