q-optimize / c3

Toolset for control, calibration and characterization of physical systems
https://c3-toolset.readthedocs.io/
Apache License 2.0
66 stars 36 forks source link

Fix Instructions ideal gate assignment in constructor #230

Closed YonatanGideoni closed 2 years ago

YonatanGideoni commented 2 years ago

What

Makes the Instruction class properly assign ideal gate inside constructor. Also fixes the two_qubits_entangling_gate.ipynb example notebook.

Why

Closes #229

How

Added self.set_ideal(ideal) to Instructions constructor.

Remarks

None

Checklist

Please include and complete the following checklist. Your Pull Request is (in most cases) not ready for review until the following have been completed. You can create a draft PR while you are still completing the checklist. Check the Contribution Guidelines for more details. You can mark an item as complete with the - [x] prefix

CLAassistant commented 2 years ago

CLA assistant check
All committers have signed the CLA.

codecov[bot] commented 2 years ago

Codecov Report

Merging #230 (fd79d3b) into dev (01cd657) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #230      +/-   ##
==========================================
+ Coverage   74.82%   74.85%   +0.02%     
==========================================
  Files          38       38              
  Lines        5534     5535       +1     
==========================================
+ Hits         4141     4143       +2     
+ Misses       1393     1392       -1     
Impacted Files Coverage Δ
c3/signal/gates.py 95.60% <100.00%> (+0.57%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

YonatanGideoni commented 2 years ago

Review was dismissed by Github because of conflict resolution, nothing important changed.

lazyoracle commented 2 years ago

@YonatanGideoni Setup the pre-commit hooks as described in the CONTRIBUTING document so that all commits are blacked and linted by default.