-
Notifications
You must be signed in to change notification settings - Fork 57
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
Qcnn docs #1275
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1275 +/- ##
=======================================
Coverage 99.94% 99.94%
=======================================
Files 73 73
Lines 10638 10638
=======================================
Hits 10632 10632
Misses 6 6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Thanks!
nclasses (int, optional): The number of classes for the classification task. Defaults to 2. | ||
params (np.ndarray, optional): The initial parameters for the QCNN. If None, random parameters are generated. Defaults to None. | ||
twoqubitansatz (qibo.models.circuit.Circuit, optional): A two-qubit ansatz for the convolutional layers. If None, a default ansatz is used. Defaults to None. | ||
copy_init_state (bool, optional): Whether to copy the initial state for each shot in the simulation. If None, the behavior depends on the backend. Defaults to None. |
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.
Fine also with this notation, but if you want you can use the [typing
] python module:
from typing import Optional
def function(nqubits: int, ..., copy_init_state: Optional[bool] = None){
...
}
Updated some docstrings in the QCNN class.
Checklist: