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

Tidy up the docs, fix typos and formatting issues #28

Merged
merged 6 commits into from
Jul 21, 2019
Merged

Conversation

josh146
Copy link
Member

@josh146 josh146 commented Jul 18, 2019

Description of the Change: resolves a few typos and formatting issues

Benefits: n/a

Possible Drawbacks: n/a

Related GitHub Issues: n/a

@josh146 josh146 requested review from nquesada and bgupt July 18, 2019 15:36
@@ -47,6 +47,9 @@
matrices with non-negative elements. This is done by sampling determinants;
the larger the number of samples taken, the higher the accuracy.

Batched hafnian algorithm
TODO
Copy link
Member Author

Choose a reason for hiding this comment

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

@nquesada or @bgupt, realised this definition was missing, could one of you guys fill it in? This is in the code API section, so just needs a short 1-2 sentence of what it does plus a link to the paper.

Something like

Returns photon number statistics of a Gaussian state for a given
covariance matrix as described in
*Multidimensional Hermite polynomials and photon distribution for polymode mixed light*
`arxiv:9308033 <https://arxiv.org/abs/hep-th/9308033>`__.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Done!

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks @nquesada!

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@codecov
Copy link

codecov bot commented Jul 18, 2019

Codecov Report

Merging #28 into master will increase coverage by 0.88%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage    95.2%   96.08%   +0.88%     
==========================================
  Files           9        9              
  Lines         625      791     +166     
==========================================
+ Hits          595      760     +165     
- Misses         30       31       +1
Impacted Files Coverage Δ
hafnian/_hafnian.py 100% <ø> (ø) ⬆️
hafnian/__init__.py 100% <ø> (ø) ⬆️
hafnian/_hermite_multidimensional.py 98% <100%> (ø) ⬆️
hafnian/quantum.py 98.44% <0%> (+0.62%) ⬆️
hafnian/samples.py 85.97% <0%> (+0.81%) ⬆️

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 ea7c2f3...7a78895. Read the comment docs.

Copy link
Contributor

@bgupt bgupt left a comment

Choose a reason for hiding this comment

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

Nice additions and improvements on the documentation. Great work!

docs/code/libhaf.rst Show resolved Hide resolved
docs/conf.py Show resolved Hide resolved
hafnian/__init__.py Show resolved Hide resolved
hafnian/_hermite_multidimensional.py Show resolved Hide resolved
src/hermite_multidimensional.hpp Show resolved Hide resolved
hafnian/__init__.py Outdated Show resolved Hide resolved
hafnian/__init__.py Outdated Show resolved Hide resolved
hafnian/__init__.py Outdated Show resolved Hide resolved
@nquesada nquesada merged commit a260a0d into master Jul 21, 2019
@nquesada nquesada deleted the doc_fixes branch July 21, 2019 12:24
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.

3 participants