Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.83%. Comparing base (b6d2a31) to head (7dcbfe3).
Report is 2 commits behind head on master.

@@           Coverage Diff           @@
##           master     #287   +/-   ##
=======================================
  Coverage   79.83%   79.83%           
=======================================
  Files          36       36           
  Lines        5429     5429           
=======================================
  Hits         4334     4334           
  Misses       1095     1095           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thierry-martinez Just wanted to kindly check in and see if you’ve had a chance to look at the PR. I’d appreciate your feedback whenever you have time

@WingCode thanks for the PR! Just want to check that entire library was scanned for the potential enhancements mentioned in the issued, copied below?

• Coherently use numpy style for docstrings throughout the library
• Use hyperlinks in the documentation, including docstrings, for better maintenability
• Check for spelling, clarity, and other enhancements in docs.

@WingCode (CC: @shinich1 )

Optionally I'm very happy if these things are covered in this PR:

• Deprecated documentation fix
  • Especially related to Clifford (previously they're passed as int to functions/methods)
• Adjust docstring styles
  • Some docstrings are not in numpy style and need fix
• Argument type fix
  • Fix documentation based on the newly-added annotations (ex. dict -> collections.abc.Mapping)
• Set default_role = any in docs/source/conf.py

Hi @shinich1 / @EarlMilktea, I've implemented the requested changes. Please let me know if there's anything else you'd like me to update or revise.

@shinich1 Happy to stared the repo and fixed the CI errors

@EarlMilktea The CI pipeline errors should be fixed now

@WingCode

Could you check and cherry-pick abc87a7 ? (I could do for you: react to this thread by 👍 )
As I warned you it seems that GitHub failed to correctly apply my suggestion.

@EarlMilktea Thank you very much for neatly organising a commit for me. I have tried to incorporated the commit abc87a7.

If any further changes is required, please feel free to go ahead and change it!