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
DM-36162: Minor documentation fixes #317
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #317 +/- ##
=======================================
Coverage 80.56% 80.56%
=======================================
Files 57 57
Lines 6389 6389
Branches 1304 1304
=======================================
Hits 5147 5147
Misses 994 994
Partials 248 248
... and 1 file with indirect coverage changes 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 in Codecov by Sentry. |
The docs claimed that these properties return a `list`, but they actually return a generator. Code that expects a list will not work.
81fbd15
to
52e1db3
Compare
@TallJimbo If you'd like, please take a look at da64ee7 to make sure I haven't garbled anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional doc fixes look good, thanks!
da64ee7
to
2b752a9
Compare
The docs have been brought in line with current DM standards, and an intersphinx link has been added for networkx types.
2b752a9
to
d3b240a
Compare
This PR fixes some minor errors in the documentation of
QuantumGraph
.