Skip to content

Conversation

@aditikashi23
Copy link

#7750

If this feature is deemed necessary, the fixes have been made: added tolerance for stabilizer effect for XPowGate, YPowGate, and ZPowGate using abs(). A similar change can be made for other gates if desired.

I am a beginner, and this is for a university class assignment to contribute to open-source software, so please let me know if this was not the intended change or something else must be fixed! I apologize in advance if that is the case.

@google-cla
Copy link

google-cla bot commented Nov 23, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions github-actions bot added the Size: XS <10 lines changed label Nov 23, 2025
@aditikashi23
Copy link
Author

I have filled out the CLA and reran the check, please let me know if there is anything else I must do. Thank you!

@codecov
Copy link

codecov bot commented Nov 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.38%. Comparing base (8dd8a8f) to head (89e1145).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7770   +/-   ##
=======================================
  Coverage   99.38%   99.38%           
=======================================
  Files        1090     1090           
  Lines       98300    98300           
=======================================
  Hits        97695    97695           
  Misses        605      605           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pavoljuhas
Copy link
Collaborator

I have filled out the CLA and reran the check, please let me know if there is anything else I must do. Thank you!

The email showing with git log --pretty=fuller on your commit is not known to the CLA. Please set your git email to the one used for the CLA (run git config user.email THE_EMAIL in cirq repo), then redo your commit with the new email (e.g., git commit --amend --reset-author) and force-push the commit.

That said, the issue #7750 is very new and at this stage it is not clear if it should be accepted. For now, please hold on here and if you'd like some other things to work on, please see the issues labeled with good-first-issue or good-for-learning. If you find an interesting one that is already assigned, but has no recent activity, feel free to add a short note you'd like to work on it and I'd be happy to reassign it your way (please add a mention @pavoljuhas so I get pinged).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants