Skip to content

refactor: Remove unnecessary casts to list#425

Merged
ValerianRey merged 1 commit intomainfrom
remove-list-cast-when-casting-to-tuple
Sep 25, 2025
Merged

refactor: Remove unnecessary casts to list#425
ValerianRey merged 1 commit intomainfrom
remove-list-cast-when-casting-to-tuple

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

@ValerianRey ValerianRey commented Sep 25, 2025

This is one less for-loop, so it should slightly improve perf.

@ValerianRey ValerianRey added the cc: refactor Conventional commit type for any refactoring, not user-facing, and not typing or perf improvements label Sep 25, 2025
@ValerianRey ValerianRey self-assigned this Sep 25, 2025
@ValerianRey ValerianRey added the cc: refactor Conventional commit type for any refactoring, not user-facing, and not typing or perf improvements label Sep 25, 2025
@ValerianRey ValerianRey changed the title refactor: remove unnecessary casts to list refactor: Remove unnecessary casts to list Sep 25, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/torchjd/autogram/_module_hook_manager.py 100.00% <100.00%> (ø)
src/torchjd/autojac/_transform/_grad.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ValerianRey ValerianRey merged commit bab9a50 into main Sep 25, 2025
16 of 17 checks passed
@ValerianRey ValerianRey deleted the remove-list-cast-when-casting-to-tuple branch September 25, 2025 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cc: refactor Conventional commit type for any refactoring, not user-facing, and not typing or perf improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant