Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CI tests for the 1.4.1 patch release [do not merge] #6653

Closed
wants to merge 9 commits into from
Closed

CI tests for the 1.4.1 patch release [do not merge] #6653

wants to merge 9 commits into from

Conversation

pavoljuhas
Copy link
Collaborator

@pavoljuhas pavoljuhas commented Jun 26, 2024 •
edited
Loading

This is a test-only PR used solely for CI testing of the Cirq-1.4.1 patch release.

@pavoljuhas
Removing 1.4.0.dev -> 1.4.0
8aa1725
@pavoljuhas
Fix expected package version in the tests
ab96766
@pavoljuhas
Improve replace_version tool and development wheels (quantumlib#6626)
9bf365e 
- Make `dev_tools/modules.py replace_version` update also the
  version expected in unit tests

- Skip type check in the new test-only source files

- Update `dev_tools/packaging/produce-package.sh` to use the `replace_version`
  utility when building development wheels
@google-cla Google CLA
Copy link

google-cla bot commented Jun 26, 2024

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.

@codecov Codecov
Copy link

codecov bot commented Jun 26, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 98.33887% with 5 lines in your changes missing coverage. Please review.

Project coverage is 97.80%. Comparing base (ab86979) to head (72e8a92).
Report is 15 commits behind head on main.

❗ Current head 72e8a92 differs from pull request most recent head 2975d10

Please upload reports for the commit 2975d10 to get more accurate results.

Files Patch % Lines
cirq-rigetti/cirq_rigetti/conftest.py 87.50% 2 Missing ⚠️
...services/quantum_engine_service/transports/grpc.py 0.00% 1 Missing ⚠️
.../quantum_engine_service/transports/grpc_asyncio.py 0.00% 1 Missing ⚠️
cirq-rigetti/cirq_rigetti/quil_input.py 99.25% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff            @@
##             main    #6653    +/-   ##
========================================
  Coverage   97.80%   97.80%            
========================================
  Files        1061     1064     +3     
  Lines       91455    91618   +163     
========================================
+ Hits        89449    89611   +162     
- Misses       2006     2007     +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

senecameeks and others added 6 commits June 26, 2024 16:31
@senecameeks @pavoljuhas
Make implicit Optional type hints in Quantum Engine PEP 484 compliant (…
e228340 
…quantumlib#6640)

* implicit optional type hints PEP 484 compliant

* rm type ignore

* add type ignore back

* mypy
@jselig-rigetti @pavoljuhas
Update cirq-rigetti to use pyquil v4 (quantumlib#6281)
6f5b4b2 
- Add support for Kraus operators, POVMs and parametric defgates
- Update quil->cirq conversion

Fixes quantumlib#6500
Partially implements quantumlib#6464
@ryanhill1 @pavoljuhas
Update requirements.txt (quantumlib#6633)
d38f24a 
Require attrs>=21.3.0 to ensure `import attrs` works.
@pavoljuhas
CI - add Python 3.12 tests to ci-daily (quantumlib#6605)
4112b59 
Run ci-daily tests with Python 3.12 in addition to 3.10 and 3.11.
Also clean up dependency constraints that got unnecessary after quantumlib#6281.

Follow-up to quantumlib#6460
@pavoljuhas
Fix CI lint failure by pinning to protobuf-4 (quantumlib#6650)
b0e5b1d 
* Pin `protobuf` to version 4 to fix "Lint check" CI failure such as
  https://github.com/quantumlib/Cirq/actions/runs/9654153610/job/26627844188

* Also bump `types-protobuf` `mypy-protobuf` for consistency with protobuf-4.x
@pavoljuhas
Bump cirq version to 1.4.1
2975d10
@pavoljuhas
Copy link
Collaborator Author

release is out, no need for this anymore.

@pavoljuhas pavoljuhas closed this Jun 27, 2024
@pavoljuhas pavoljuhas deleted the v1.4.1-dev branch June 27, 2024 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vtomole vtomole Awaiting requested review from vtomole vtomole will be requested when the pull request is marked ready for review vtomole is a code owner

@HGSilveri HGSilveri Awaiting requested review from HGSilveri HGSilveri will be requested when the pull request is marked ready for review HGSilveri is a code owner

@ma5x ma5x Awaiting requested review from ma5x ma5x will be requested when the pull request is marked ready for review ma5x is a code owner

@pschindler pschindler Awaiting requested review from pschindler pschindler will be requested when the pull request is marked ready for review pschindler is a code owner

@dabacon dabacon Awaiting requested review from dabacon dabacon will be requested when the pull request is marked ready for review dabacon is a code owner

@ColemanCollins ColemanCollins Awaiting requested review from ColemanCollins ColemanCollins will be requested when the pull request is marked ready for review ColemanCollins is a code owner

@nakardo nakardo Awaiting requested review from nakardo nakardo will be requested when the pull request is marked ready for review nakardo is a code owner

@gmauricio gmauricio Awaiting requested review from gmauricio gmauricio will be requested when the pull request is marked ready for review gmauricio is a code owner

@wcourtney wcourtney Awaiting requested review from wcourtney wcourtney will be requested when the pull request is marked ready for review wcourtney is a code owner

@verult verult Awaiting requested review from verult verult will be requested when the pull request is marked ready for review verult is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pavoljuhas @senecameeks @jselig-rigetti @ryanhill1