Fixes #7072: Support more OpenQASM 2.0 gates from qelib1.inc

[ACE07-Sev]

• Fixes Support more OpenQASM 2.0 gates from qelib1.inc #7072

Added:

• u0
• u
• p
• csx (via ops.ControlledGate)
• cu2 (via ops.ControlledGate)
• cu (via ops.ControlledGate)
• c3x (via ops.ControlledGate)
• c4x (via ops.ControlledGate)

Added some preliminary testers for maintainer review and feedback.

[google-cla]

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.

[ACE07-Sev]

I see the format issue, but why did Ubuntu fail?

[pavoljuhas]

@ACE07-Sev - please complete the CLA, we are not able to accept PRs without it.

[ACE07-Sev]

Done sir.

[ACE07-Sev]

It still fails with Ubuntu it seems.

[ACE07-Sev]

Found the issues. Pushing in a moment.

[ACE07-Sev]

Some gates from the ones that I added and the ones the last person added (crx, cry) were missing in tester params. It should be okay now.

I'm surprised why Windows and MacOs passed.

[ACE07-Sev]

@mhucka Apologies for the bother. Could you kindly run the actions please?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.68%. Comparing base (c000fae) to head (8f81a93).
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #7377   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files        1112     1112           
  Lines       97641    97678   +37     
=======================================
+ Hits        96354    96396   +42     
+ Misses       1287     1282    -5     

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

[daxfohl]

@ACE07-Sev fyi you can run check/format-incremental --apply from command line. I usually run that, check/pytest-changed-files, check/mypy, check/pylint-changed-files, then finally check/pytest before submitting.

[ACE07-Sev]

@daxfohl I tried that, but it opens the code file for the formatter instead of formatting the .py. Maybe I'm doing sth wrong. The difference is small, I can quickly fix from the traceback and push for now.

[ACE07-Sev]

I see it. Fixing now...

I do apologize for the inconvenience by the way. I usually run these locally, but couldn't set them up properly.

[ACE07-Sev]

Test pwease?

[ACE07-Sev]

Is it being sarcastic by "short test summary" for the one that failed? That's so long hehe.

[ACE07-Sev]

The one that failed seems to be from cirq-web. I don't think that's sth I broke, maybe it's from what you were discussing yesterday during the biweekly session?

[ACE07-Sev]

I'd appreciate some feedback on the PR and guidance for the error. Thank you.

[ACE07-Sev]

I see the checks pass now. LGTY?

[ACE07-Sev]

Greetings there,

May I ask if there are any other items I need to add/modify for the PR?