Skip to content

refactor(autogram): Rework VJP class hierarchy#433

Merged
ValerianRey merged 1 commit intomainfrom
add-Vmapped
Sep 29, 2025
Merged

refactor(autogram): Rework VJP class hierarchy#433
ValerianRey merged 1 commit intomainfrom
add-Vmapped

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

No description provided.

@ValerianRey ValerianRey added cc: refactor Conventional commit type for any refactoring, not user-facing, and not typing or perf improvements package: autogram labels Sep 29, 2025
@ValerianRey ValerianRey self-assigned this Sep 29, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 29, 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/autogram/_vjp.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

* Split VJP base class into VJP (for the __call__ prototype) and
  ModuleVJP (for referencing a module, its trainable and frozen
parameters).
* Add Vmapped to wrap a VJP
* Remove VJPType (now useless because a vmapped VJP will also be of type
  VJP).
@ValerianRey ValerianRey changed the title refactor(autogram): Add Vmapped(VJP) refactor(autogram): Rework VJP class hierarchy Sep 29, 2025
@ValerianRey ValerianRey merged commit 170e7c5 into main Sep 29, 2025
17 checks passed
@ValerianRey ValerianRey deleted the add-Vmapped branch September 29, 2025 15:05
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 package: autogram

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant