Skip to content

Conversation

@dwierichs
Copy link
Contributor

Context:
Adjoints are not handled correctly, see linked issue

Description of the Change:
Handle them correctly.

Benefits:
Bug fix

Possible Drawbacks:
N/A

Related GitHub Issues:
[sc-100982]
fixes #8428

@dwierichs dwierichs added this to the v0.43 milestone Oct 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (v0.43.0-rc0@9439855). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff               @@
##             v0.43.0-rc0    #8429   +/-   ##
==============================================
  Coverage               ?   99.42%           
==============================================
  Files                  ?      584           
  Lines                  ?    61250           
  Branches               ?        0           
==============================================
  Hits                   ?    60898           
  Misses                 ?      352           
  Partials               ?        0           

☔ 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.

@dwierichs dwierichs added the review-ready 👌 PRs which are ready for review by someone from the core team. label Oct 8, 2025
@dwierichs dwierichs linked an issue Oct 8, 2025 that may be closed by this pull request
1 task
@dwierichs dwierichs requested a review from mehrdad2m October 8, 2025 18:55
Copy link
Contributor

@mehrdad2m mehrdad2m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Great! Thanks for catching and fixing this bug! 🚀

@dwierichs dwierichs merged commit 19c4d6c into v0.43.0-rc0 Oct 8, 2025
49 checks passed
@dwierichs dwierichs deleted the merge-rots-xdsl-adjoint branch October 8, 2025 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-ready 👌 PRs which are ready for review by someone from the core team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] merge_rotations_pass does not take adjoint correctly into account

5 participants