unvectorization function

[renatomello]

Checklist:

• Reviewers confirm new code works as expected.
• Tests are passing.
• Coverage does not decrease.
• Documentation is updated.

[renatomello]

This closes #738 . I generalized the vectorization function to include row- and column-wise vectorization, and created an unvectorization function for these two methods. For now, system-wise unvectorization is raising NotImplementedError.

[codecov]

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage 👍

Coverage data is based on head (840d3ca) compared to base (9788edf).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #743   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           94        94           
  Lines        12390     12439   +49     
=========================================
+ Hits         12390     12439   +49     

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/quantum_info/basis.py	100.00% <100.00%> (ø)
src/qibo/tests/test_quantum_info_basis.py	100.00% <100.00%> (ø)

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[renatomello]

This closes #738 . I generalized the vectorization function to include row- and column-wise vectorization, and created an unvectorization function for these two methods. For now, system-wise unvectorization is raising NotImplementedError.

Now there's an implementation of unvectorization(order="system") thanks to Raghav.

[scarrazza]

@renatomello thank you, looks good to me. @stavros11 could you please double check before we merge?

[stavros11]

@renatomello thank you, looks good to me. @stavros11 could you please double check before we merge?

Yes, looks good to me and tests are passing including GPU. I think we can merge, thanks.