Skip to content

chore: ty#306

Open
jonasteuwen wants to merge 3 commits into
mainfrom
chore/ty
Open

chore: ty#306
jonasteuwen wants to merge 3 commits into
mainfrom
chore/ty

Conversation

@jonasteuwen
Copy link
Copy Markdown
Contributor

Modifications to make ty pass + workflow.

@jonasteuwen jonasteuwen self-assigned this May 21, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

Codecov Report

❌ Patch coverage is 84.05316% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.73%. Comparing base (c12e041) to head (01b60df).

Files with missing lines Patch % Lines
direct/functionals/hfen.py 61.53% 10 Missing ⚠️
direct/inference.py 0.00% 9 Missing ⚠️
direct/utils/io.py 14.28% 6 Missing ⚠️
direct/engine.py 75.00% 4 Missing ⚠️
direct/common/subsample.py 89.65% 3 Missing ⚠️
direct/data/datasets.py 89.65% 3 Missing ⚠️
direct/train.py 62.50% 3 Missing ⚠️
direct/cli/utils.py 0.00% 2 Missing ⚠️
direct/nn/mobilenet/mobilenet.py 0.00% 2 Missing ⚠️
direct/data/mri_transforms.py 90.90% 1 Missing ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #306      +/-   ##
==========================================
- Coverage   85.80%   85.73%   -0.08%     
==========================================
  Files         103      103              
  Lines        8968     9041      +73     
==========================================
+ Hits         7695     7751      +56     
- Misses       1273     1290      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Squashed branch work:
- fix: make ty (type checker) pass across the codebase
- chore: update toolchain (ruff, formatters)
- chore: update/fix bazel deps and tests
- chore: ignore ruff outside of direct
- fix: code coverage workflow (uv/pytest)
- fix: remove masks

Co-authored-by: Cursor <cursoragent@cursor.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