-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Increase documentation of ParameterVector
and ParameterVectorElement
#13761
Conversation
These were somewhat lacking before, despite being fairly heavily used. `ParameterVectorElement` was not publicly documented, despite adding additional properties that were intended for public use.
One or more of the following people are relevant to this code:
|
Pull Request Test Coverage Report for Build 13054802059Details
💛 - Coveralls |
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.
LGTM but I won't put it on the merge queue since Luciano is assigned 🙂
Oh, Luciano's just assigned because this just came out of a meeting he and I were in together. It was more an FYI, and I assumed others were busy. |
In that case.... |
…nt` (#13761) (#13771) These were somewhat lacking before, despite being fairly heavily used. `ParameterVectorElement` was not publicly documented, despite adding additional properties that were intended for public use. (cherry picked from commit d03a61e) Co-authored-by: Jake Lishman <[email protected]>
…nt` (Qiskit#13761) These were somewhat lacking before, despite being fairly heavily used. `ParameterVectorElement` was not publicly documented, despite adding additional properties that were intended for public use.
Summary
These were somewhat lacking before, despite being fairly heavily used.
ParameterVectorElement
was not publicly documented, despite adding additional properties that were intended for public use.Details and comments