-
Notifications
You must be signed in to change notification settings - Fork 322
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
[Stable] Prepare 0.2.1 release #193
Merged
manoelmarques
merged 9 commits into
qiskit-community:stable/0.2
from
manoelmarques:stable-release
Aug 24, 2021
Merged
[Stable] Prepare 0.2.1 release #193
manoelmarques
merged 9 commits into
qiskit-community:stable/0.2
from
manoelmarques:stable-release
Aug 24, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The max circuits per job for most/all backend devices is 900. Setting batch_size=900 fixes an inefficient jobs being created. Co-authored-by: Anton Dekusar <adekusar@ie.ibm.com> Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com>
…t-community#112) (qiskit-community#151) * Added a wrapper `get_objective` method in `TrainableModel`, and `callback` argument to NN classifier `get_objective` returns a callable which is passed as objective function to NN classifier. It checks for a callable `callback` argument, which if not `None` can access the intermediate data during the optimization. * style changes * Updated callback tests for `NeuralNetworkClassifier` * Added callback for `NeuralNetworkRegressor` * Added tests for callback * Refactored `get_objective` * Added release note * Updated `callback` argument docstring * Shifted `callback` parameter to parent class `TrainableModel` * Added assertions for number of weights in `TestNeuralNetworkClassifier` `TestNeuralNetworkRegressor` * Rephrased release note and debugged tests for callback Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com> Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com>
…unity#62) (qiskit-community#143) Added categorical label encoding * Added documentation * Update qiskit_machine_learning/algorithms/classifiers/neural_network_classifier.py * Update releasenotes/notes/cateforical-output-049e682adc9d1e28.yaml * Update releasenotes/notes/cateforical-output-049e682adc9d1e28.yaml * Update releasenotes/notes/cateforical-output-049e682adc9d1e28.yaml * Cleaned and organized code * Update qiskit_machine_learning/algorithms/classifiers/neural_network_classifier.py * Update qiskit_machine_learning/algorithms/classifiers/neural_network_classifier.py * Update qiskit_machine_learning/algorithms/classifiers/neural_network_classifier.py * Split categorical encoding into two functions for classifier's fit and score methods * Set sparse=False in one-hot encoder * removed unnecessary copy of arrays * Renamed methods and fixed label reshape * Added explanatory comments Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com> Co-authored-by: Anton Dekusar <adekusar@ie.ibm.com> Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com>
* Add hybrid qnn unit tests * Fix spelling * Add docstrings * Retry * Fix style * Register weight name * Update test * Separate hybrid tests * Fix spelling * Fix lint * Add comments * Remove unused functions * Add register weight param. reno * Clarify procedure * Remove unused import Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com> Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com>
…ty#179) Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
…skit-community#79) * added getter and setter, raises exception if no qi * set sampler in qi setter, added tests * added logic to setters, fixed tests * fix black * calling setter with instantiation * setters no longer accept optional qi * undid changes to docstrings * reset output shape within qi setter * revised implementation * fix spell * fix mypy * Update qiskit_machine_learning/neural_networks/circuit_qnn.py Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> * Update qiskit_machine_learning/neural_networks/circuit_qnn.py Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> * Update qiskit_machine_learning/neural_networks/opflow_qnn.py Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> * code review * updated docstrings * Update qiskit_machine_learning/neural_networks/circuit_qnn.py Co-authored-by: dlasecki <dal@zurich.ibm.com> * Update qiskit_machine_learning/neural_networks/circuit_qnn.py Co-authored-by: dlasecki <dal@zurich.ibm.com> * code review * docstrings Co-authored-by: Anton Dekusar <adekusar@ie.ibm.com> Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com> Co-authored-by: Manoel Marques <Manoel.Marques@ibm.com> Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> Co-authored-by: dlasecki <dal@zurich.ibm.com>
* Made TrainableModel use global seed (issue qiskit-community#158) Made TrainableModel use RandomState object seeded by algorithm_globals.random_seed to choose an initial point for the optimizer outside of warm start. * Global random seed for QSVC constructor (issue qiskit-community#158) The value of algorithm_globals.random_seed is being used as random_state parameter for SVC constructor during construction of QSVC object. * Added Qiskit into requirements.txt (issu qiskit-community#158) Added Qiskit~=0.26.0 into requirements.txt, as qiskit.utils contain algorithm_globals object. * Added Qiskit into requirements.txt & improved code formatting (issue qiskit-community#158) Added Qiskit~=0.26.0 into requirements.txt, as qiskit.utils contain algorithm_globals object. * Removed unnecessary explicit requirement of Qiskit package (issue qiskit-community#158). * Simplified random number generation using global seed (issue qiskit-community#158). * random_state is now being randomly initialized only if not passed as parameter (issue qiskit-community#158). * Parameter 'random_state' now initialized by 'random_seed' in kwargs (issue qiskit-community#158). * Update qiskit_machine_learning/algorithms/trainable_model.py * Reverted incorrect changes to requirements.txt (issue qiskit-community#158). Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com> Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
* Fix num_qubits * Remove _num_qubits * Fix style * Add tests for copy, bind_parameters * Add Fix RawFeatVec reno * Fix typing * Fix test * Add reset registers * Fix num_qubits>0 * Fix mypy conflict * Fix black * Update if check Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> * Add if check * Add comment * Modify reno * code review * code review * updated docstrings * code review Co-authored-by: Anton Dekusar <62334182+adekusar-drl@users.noreply.github.com> Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com> Co-authored-by: Anton Dekusar <adekusar@ie.ibm.com>
manoelmarques
requested review from
adekusar-drl,
pbark and
stefan-woerner
as code owners
August 24, 2021 11:50
adekusar-drl
approved these changes
Aug 24, 2021
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!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Release 0.2.1 backporting #150, #112, #62, #156, #179, #79, #178, #182
Details and comments