Skip to content

Add CI pipeline, unit tests, bug fixes, and contributing docs#8

Open
shreyaskamathkm wants to merge 1 commit intodevelopfrom
claude/wonderful-lamarr
Open

Add CI pipeline, unit tests, bug fixes, and contributing docs#8
shreyaskamathkm wants to merge 1 commit intodevelopfrom
claude/wonderful-lamarr

Conversation

@shreyaskamathkm
Copy link
Copy Markdown
Owner

Bug fixes:

  • Fix misleading validator error message in cfg_dataclasses.py
  • Replace print() with logger in model_helpers.py
  • Fix hard-coded Path.cwd() in resnetv1b.py (now package-relative, overridable via FTNET_PRETRAINED_DIR)

Tooling:

  • Add mypy, pytest, pytest-cov to dev dependencies in pyproject.toml
  • Add [tool.mypy] and [tool.pytest.ini_options] config sections
  • Add mypy pre-commit hook

CI/CD:

  • Add .github/workflows/ci.yml with lint, type-check, and test jobs (Python 3.10 & 3.11)

Tests:

  • Add tests/test_cfg_dataclasses.py
  • Add tests/test_loss.py
  • Add tests/test_model_helpers.py
  • Add tests/test_schedulers.py
  • Add tests/test_data_attributes.py

Docs:

  • Add CONTRIBUTING.md with branch strategy, setup, and PR guidelines
  • Add GitHub issue templates (bug report, feature request)
  • Add GitHub PR template with checklist targeting develop

Bug fixes:
- Fix misleading validator error message in cfg_dataclasses.py
- Replace print() with logger in model_helpers.py
- Fix hard-coded Path.cwd() in resnetv1b.py (now package-relative, overridable via FTNET_PRETRAINED_DIR)

Tooling:
- Add mypy, pytest, pytest-cov to dev dependencies in pyproject.toml
- Add [tool.mypy] and [tool.pytest.ini_options] config sections
- Add mypy pre-commit hook

CI/CD:
- Add .github/workflows/ci.yml with lint, type-check, and test jobs (Python 3.10 & 3.11)

Tests:
- Add tests/test_cfg_dataclasses.py
- Add tests/test_loss.py
- Add tests/test_model_helpers.py
- Add tests/test_schedulers.py
- Add tests/test_data_attributes.py

Docs:
- Add CONTRIBUTING.md with branch strategy, setup, and PR guidelines
- Add GitHub issue templates (bug report, feature request)
- Add GitHub PR template with checklist targeting develop

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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