Skip to content

refactor(autogram): Rename internal variables in VJP#435

Merged
ValerianRey merged 2 commits intomainfrom
rename-vjp-variables
Sep 29, 2025
Merged

refactor(autogram): Rename internal variables in VJP#435
ValerianRey merged 2 commits intomainfrom
rename-vjp-variables

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

@ValerianRey ValerianRey commented Sep 29, 2025 •
edited
Loading

  • Rename primals to trainable_params in functional_model_call
  • Rename inputs to args

@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 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
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.

@ValerianRey ValerianRey merged commit b542e9f into main Sep 29, 2025
17 checks passed
@ValerianRey ValerianRey deleted the rename-vjp-variables branch September 29, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PierreQuinton PierreQuinton PierreQuinton approved these changes

Assignees

@ValerianRey ValerianRey

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ValerianRey @PierreQuinton