Add attrs lower bound in requirements.txt

[ryanhill1]

$ pip install cirq-core
$ pip install 'attrs==21.2.0'
$ python
>>> import cirq
...
File ~/anaconda3/envs/py312/lib/python3.12/site-packages/cirq/transformers/gauge_compiling/gauge_compiling.py:23
     20 import functools
     22 from dataclasses import dataclass
---> 23 from attrs import frozen, field
     24 import numpy as np
     26 from cirq.transformers import transformer_api

ModuleNotFoundError: No module named 'attrs'

$ pip install 'attrs==21.3.0'
$ python
>>> import cirq
>>> # works

Any attrs<21.3.0 fails for me when I try to import cirq. And any attrs>=21.3.0 seems to work.

Reference:

• https://pypi.org/project/attrs/#history
• https://stackoverflow.com/a/75610797/14781553

[pavoljuhas]

Unfortunately, this breaks the pip dependency resolver and the ci tests even whilst uv pip compile dev_tools/requirements/dev.env.txt can find a viable set of packages.

Installation of cirq-core to a fresh virtual environment brings in the latest attrs-23.2.0.
Installation of the whole cirq suite brings attrs-21.4.0 - both of these cases satisfy the required minimum version.

I suggest to hold this for later until after the merge of #6281 which will loosen the maximum attrs version from a transitive dependency.

[NoureldinYosri]

I don't see how ModuleNotFoundError: No module named 'attrs' could be caused by an old attrs version?! ... if the error was that it can't find one of the functions/classes of the library then it would have made sense ... but module not found is a different thing

[maffoo]

@NoureldinYosri, see the attrs changelog. While the package on pypi has always been called attrs so that is what you would install (pip install attrs), the python package that you would import was previously called attr (import attr). This was confusing, so they added an attrs python package which matches the install name, and this first landed in 21.3.0.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.81%. Comparing base (5f7881f) to head (f55d42e).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6633      +/-   ##
==========================================
- Coverage   97.81%   97.81%   -0.01%     
==========================================
  Files        1067     1066       -1     
  Lines       91547    91693     +146     
==========================================
+ Hits        89546    89687     +141     
- Misses       2001     2006       +5     

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

[pavoljuhas]

LGTM, thank you!