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.

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

Update cirq version and citation #714

Merged
merged 2 commits into from Mar 7, 2021
Merged

Update cirq version and citation #714

merged 2 commits into from Mar 7, 2021

Conversation

ncrubin
Copy link
Collaborator

@ncrubin ncrubin commented Mar 6, 2021

Citation now references published paper. Cirq version is now pinned to new release.

Citation now references published paper.  Cirq version is now pinned to new release.
@google-cla google-cla bot added the cla: yes label Mar 6, 2021
param_resolver now takes a recursive boolean.

hat-tip to Balint for pointing this out and saving a bunch of time!
@ncrubin ncrubin merged commit 10637da into master Mar 7, 2021
@ncrubin ncrubin deleted the cirq_version_bump branch March 7, 2021 00:21
@mpharrigan
Copy link
Collaborator

cc @balopat @rmlarose we really need to disentangle the requirements.txt file from install_requires. This causes issues for packages that depend both on cirq and openfermion. Is it actually broken if you use new openfermion with cirq 0.9?

ncrubin added a commit to ncrubin/OpenFermion that referenced this pull request Jul 25, 2022
* Update cirq version and citation

Citation now references published paper.  Cirq version is now pinned to new release.

* Param-resolve breaking change in cirq0.10.0

param_resolver now takes a recursive boolean.

hat-tip to Balint for pointing this out and saving a bunch of time!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants