Skip to content

Switch CI type checking to ty and fix typing issues - #2

Open
RajeevAtla wants to merge 2 commits into
mainfrom
codex/run-ruff-check-and-fix-errors
Open

Switch CI type checking to ty and fix typing issues#2
RajeevAtla wants to merge 2 commits into
mainfrom
codex/run-ruff-check-and-fix-errors

Conversation

@RajeevAtla

Copy link
Copy Markdown
Owner

Summary

  • replace the CI mypy step with ty and install the new checker alongside existing tooling
  • harden optional torch imports across models, training helpers, demos, and scripts with TYPE_CHECKING shims and safe fallbacks that satisfy ty
  • tighten dataset utilities, evaluation metrics, and tests with clearer type annotations and runtime conversions uncovered by the new checker

Testing

  • ty check --python $(python -c 'import sys; print(sys.executable)')
  • ruff check .

https://chatgpt.com/codex/tasks/task_e_68f5cac3a4048328b8424391f9555a7d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant