Skip to content

Conversation

@groberts-flex
Copy link
Contributor

@groberts-flex groberts-flex commented Feb 11, 2026

Optional argument for when this is imported by the backend for the adjoint pipeline

updated emulated_run in unit test to catch a regression for this

(reposting because of error in branch name for jira)


Note

Low Risk
Low risk: a small signature/default-argument change plus a test update, with no change to adjoint computation logic beyond allowing omitted custom_vjp.

Overview
Adjoint postprocessing no longer requires custom_vjp. postprocess_adj now defaults custom_vjp to None, allowing callers (including backend imports) to omit the parameter.

Tests updated to prevent regression. The autograd emulation unit test stops passing custom_vjp=None explicitly, ensuring the defaulted call path is exercised.

Written by Cursor Bugbot for commit d1ef0f7. This will update automatically on new commits. Configure here.

@groberts-flex groberts-flex force-pushed the groberts-flex/FXC-5606-fix-postprocess-adj-custom-vjp branch from 6d97fbd to 0ab0117 Compare February 11, 2026 19:42
@github-actions
Copy link
Contributor

Diff Coverage

Diff: origin/develop...HEAD, staged and unstaged changes

No lines with coverage information in this diff.

@groberts-flex groberts-flex force-pushed the groberts-flex/FXC-5606-fix-postprocess-adj-custom-vjp branch from 0ab0117 to d1ef0f7 Compare February 11, 2026 21:03
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