Add daily CI test for a fresh cirq --pre installation

[pavoljuhas]

Description of the issue

pip install cirq --pre may break with cirq installation if some of cirq dependencies release a dev package with API changes to PyPI. The --pre option seems to affect cirq as well as any of its requirements. See #6129 for an example breakage.

Cirq version

v0.15.0-298-g24abfa1d

Proposed solution

Add a daily CI test to perform a fresh pip install --pre cirq from PyPI and execute unit tests on the installed code.

This requires updates to our pytest configuration; as things are pytest requires the ./conftest.py from the Cirq repository root, but this file is not distributed in cirq wheels.

[jackreeceejini]

I'm trying to work on this issue, any pointers on how to start?

[pavoljuhas]

I'd suggest to add a new CI workflow ci-daily.yml scheduled to run every day, see #6046 for an example.
Then you can use the Pytest Ubuntu, Pytest Windows, Pytest MacOs jobs as starting points,
but modify their Install requirements parts to do

$ pip install --pre cirq

instead of installing from the dev.env.txt package list.

I think you will also need to do

pip install \
    -r dev_tools/requirements/deps/format.txt \
    -r dev_tools/requirements/deps/pylint.txt \
    -r dev_tools/requirements/deps/pytest.txt

to be able to run check/pytest at line 171

Cirq/.github/workflows/ci.yml

Lines 170 to 171 in 6c251a3

	- name: Pytest check
	run: check/pytest -n auto --ignore=cirq-core/cirq/contrib --rigetti-integration

The idea is to install the pre-release cirq wheel and its dependencies from PyPI and verify things still work.

[jackreeceejini]

Okay. Thanks

…

On Fri, Jun 23, 2023, 5:36 PM Pavol Juhas ***@***.***> wrote: I'd suggest to add a new CI workflow ci-daily.yml scheduled to run every day, see #6046 <#6046> for an example. Then you can use the Pytest Ubuntu, Pytest Windows, Pytest MacOs jobs as starting points, but modify their Install requirements <https://github.com/quantumlib/Cirq/blob/6c251a3dcf30cc89208939012fe6fed82d004d67/.github/workflows/ci.yml#L164-L167> parts to do $ pip install --pre cirq instead of installing from the dev.env.txt package list. I think you will also need to do pip install \ -r dev_tools/requirements/deps/format.txt \ -r dev_tools/requirements/deps/pylint.txt \ -r dev_tools/requirements/deps/pytest.txt to be able to run check/pytest at line 171 https://github.com/quantumlib/Cirq/blob/6c251a3dcf30cc89208939012fe6fed82d004d67/.github/workflows/ci.yml#L170-L171 The idea is to install the pre-release cirq wheel and its dependencies from PyPI and verify things still work. — Reply to this email directly, view it on GitHub <#6130 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHBN3VVRUBUDDFDBCP3YKPDXMYD5VANCNFSM6AAAAAAZBB2XQ4> . You are receiving this because you commented.Message ID: ***@***.***>