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

[FIX] Allow parametrized phase in EOM mode #599

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

HGSilveri
Copy link
Collaborator

I stumbled upon this bug when trying to make EOM pulses with a parametrized phase.

@HGSilveri HGSilveri requested a review from a-corni October 18, 2023 16:35
Copy link
Collaborator

@a-corni a-corni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find ! Looks good to me :)

@HGSilveri HGSilveri merged commit 581886f into develop Oct 19, 2023
6 checks passed
@HGSilveri HGSilveri deleted the hs/fix-eom-param-phase branch October 19, 2023 08:36
a-corni added a commit that referenced this pull request Nov 16, 2023
* [FIX] Allow parametrized phase in EOM mode (#599)

* Parametrized pulse in EOM fix

* Restrict to mypy 1.1.1

* Move to 1.3.0

* Fix pydantic

* Blocking type checking of referencing.py

* skipping imports for referencing

* silenting import errors with referencing

* Skip referencing in mypy

---------

Co-authored-by: Henrique Silvério <[email protected]>
@HGSilveri HGSilveri mentioned this pull request Dec 1, 2023
HGSilveri pushed a commit that referenced this pull request Dec 4, 2023
Main changes:
a0c59a6 Deprecate legacy devices (#620)
b18e434 FIX: Detuning modulation for custom EOM buffer time (#619)
8c5d966 Delete summing constraint in DetuningMap (#610)
b41d62a JSON serialization support for numpy integer types (#617)
9f9a17a Fix temperature initialization in SimConfig (#606)
2bad427 Fix spacing in register.max_connectivity (#605)
581886f [FIX] Allow parametrized phase in EOM mode (#599)
394fe6a Add `wait` option to remote backends (#598)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants