Skip to content

Clean up preprocess and pass correct gate set#1341

Merged
astralcai merged 23 commits intomasterfrom
gate-set
Feb 12, 2026
Merged

Clean up preprocess and pass correct gate set#1341
astralcai merged 23 commits intomasterfrom
gate-set

Conversation

@astralcai
Copy link
Contributor

Context:

  • Update the preprocess transforms to pass the correct gate set to decompose.
  • Clean up duplicate code in the preprocess transforms.

Description of the Change:

Benefits:

Possible Drawbacks:

Related GitHub Issues:
[sc-110772]

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Hello. You may have forgotten to update the changelog!
Please edit .github/CHANGELOG.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 Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.62%. Comparing base (7d89251) to head (93ea3b3).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1341      +/-   ##
==========================================
+ Coverage   95.79%   96.62%   +0.83%     
==========================================
  Files         243      307      +64     
  Lines       40533    47342    +6809     
==========================================
+ Hits        38829    45746    +6917     
+ Misses       1704     1596     -108     
Flag Coverage Δ
unit_tests 96.62% <100.00%> (+0.83%) ⬆️

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

@blacksmith-sh

This comment has been minimized.

@astralcai astralcai changed the title [WIP] Clean up preprocess and pass correct gate set Clean up preprocess and pass correct gate set Feb 6, 2026
@LuisAlfredoNu LuisAlfredoNu added ci:use-multi-gpu-runner Enable usage of Multi-GPU runner for this Pull Request ci:build_wheels Activate wheel building. ci:use-gpu-runner Enable usage of GPU runner for this Pull Request labels Feb 9, 2026
@LuisAlfredoNu
Copy link
Contributor

Thanks @astralcai
Could you trigger the CIs again to check that the LG and LK pass the tests, please?

@blacksmith-sh

This comment has been minimized.

Copy link
Contributor

@LuisAlfredoNu LuisAlfredoNu left a comment

Choose a reason for hiding this comment

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

Thank you @astralcai
LGTM

Copy link
Member

@maliasadi maliasadi left a comment

Choose a reason for hiding this comment

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

LGTM! My only remaining concern #1341 (comment) is unrelated to this PR and requires further discussion.

@astralcai astralcai merged commit 7fe0771 into master Feb 12, 2026
183 checks passed
@astralcai astralcai deleted the gate-set branch February 12, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:build_wheels Activate wheel building. ci:use-gpu-runner Enable usage of GPU runner for this Pull Request ci:use-multi-gpu-runner Enable usage of Multi-GPU runner for this Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants