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

Full state callback #559

Merged
merged 5 commits into from
Mar 16, 2022
Merged

Full state callback #559

merged 5 commits into from
Mar 16, 2022

Conversation

stavros11
Copy link
Member

Implements a callback that keeps track of the full state vector or density matrix during circuit simulation. This was requested by @igres26 in #555. Usage is similar to existing callbacks, one can add this to a circuit, in the place which a stamp of the state is required, using a CallbackGate.

This PR is complete, I just mark it as do not merge because the test will change if we merge qiboteam/qibojit#74.

@stavros11 stavros11 added the do not merge Blocked by some other development or PR label Mar 10, 2022
@stavros11 stavros11 requested a review from scarrazza March 10, 2022 10:37
@codecov
Copy link

codecov bot commented Mar 16, 2022

Codecov Report

Merging #559 (348c2f3) into master (e2201c7) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #559   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           84        84           
  Lines        12651     12681   +30     
=========================================
+ Hits         12651     12681   +30     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/qibo/abstractions/callbacks.py 100.00% <100.00%> (ø)
src/qibo/core/callbacks.py 100.00% <100.00%> (ø)
src/qibo/tests/test_core_callbacks.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2201c7...348c2f3. Read the comment docs.

@stavros11 stavros11 removed the do not merge Blocked by some other development or PR label Mar 16, 2022
@stavros11
Copy link
Member Author

@scarrazza I updated the test after the latest merge in qibojit. This should be good to go.

@scarrazza
Copy link
Member

Thanks!

@scarrazza scarrazza merged commit e66224d into master Mar 16, 2022
@stavros11 stavros11 deleted the statecallback branch March 27, 2022 07:57
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