Fix conflicts #11673
Merged
Fix conflicts #11673
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
|
One or more of the the following people are requested to review this:
|
ElePT
merged commit 2d13f7e
into
Qiskit:mergify/bp/stable/0.46/pr-10266
2 of 6 checks passed
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jan 30, 2024
* Add fake generic and modify tests (#10266) * Added a FakeGeneric BackendV2 backend, this is just a bare minimum working code * All FakeBackends (like FakeMelbourne, FakeBoeblingen ,etc) is replaced with FakeGeneric * relocated FakeGeneric in fake_provider directory; replaced if-elif with instruction_dict in fake_generic.py; added support of grid type of coupling map; using lower number of qubits, passed all tests of test_transpiler except one method ( test_parallel_dispatch_lazy_cal_loading ) * added tests for FakeGeneric * This commit just reformats test_transpiler.py fake_generic.py and test_fake_generic.py * Update qiskit/providers/fake_provider/fake_generic.py Co-authored-by: atharva-satpute <55058959+atharva-satpute@users.noreply.github.com> * Update qiskit/providers/fake_provider/fake_generic.py Co-authored-by: atharva-satpute <55058959+atharva-satpute@users.noreply.github.com> * Update test/python/providers/fake_provider/test_fake_generic.py Co-authored-by: atharva-satpute <55058959+atharva-satpute@users.noreply.github.com> * Update test/python/providers/fake_provider/test_fake_generic.py Co-authored-by: atharva-satpute <55058959+atharva-satpute@users.noreply.github.com> * Fix some tests * Add calibration schedules, fix transpiler tests * Fix lint * Refactor, add pulse functionality * Latest updates to FakeGeneric * Fix some unit tests * Fix some unit tests * Fix tests, lint, refactor * Remove unused inputs * Make calibrations optional * Attempt to speed up test: only add calibrations if option enabled * Revert some tests to V1 * Refactor FakeGeneric * Update unit test * Rename supported instructions to operations, fix tests Fix test seed Fix unit test, lint * Restore vf2postlayout test to V1 Fx vf2 test * Add pulse test to validate pulse capabilities of new fake backend Update fake generic * Update docs, fix lint * Avoid set for basis gates to allow reproducibility in the error/duration value generation * Update calibrations * Fix transpiler test * Add run test to confirm noise defaults * Add CZ to basis gates, dtm, update backend names, fix docs * Remove GenericTarget, add default for basis_gates, remove supported_operations. * Apply review comments, modify calibrate_instructions to avoid public calibration method. * New name proposal * Fix lint and test * Fix comment * Apply Kevin's suggestions to extract defaults from class * Make defaults private * Fix lint * Revert previous 3 comments. * Privatize GenericFakeBackend * Remove from docs * Revert privatization * Apply review comments * Add reno * Fix lint * Apply suggestions from Matt's code review Co-authored-by: Matthew Treinish <mtreinish@kortar.org> * Apply comments from code review * Fix lint * Rename to GenericBackendV2 and adjust docs. * Fix lint * Apply comments from code review * It's not 2023 anymore * Fix conflict * Fix test conflict. --------- Co-authored-by: atharva-satpute <55058959+atharva-satpute@users.noreply.github.com> Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com> Co-authored-by: Elena Peña Tapia <epenatap@gmail.com> Co-authored-by: Matthew Treinish <mtreinish@kortar.org> Co-authored-by: Kevin Hartman <kevin@hart.mn> (cherry picked from commit b4582a9) # Conflicts: # qiskit/providers/fake_provider/__init__.py # test/python/compiler/test_transpiler.py * Update __init__.py * Update test_transpiler.py * Update test_transpiler.py * Fix conflicts (#11673) * Rename test, switch to basic simulator * Fix bug in pulse default generation --------- Co-authored-by: MozammilQ <54737489+MozammilQ@users.noreply.github.com> Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>
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
Details and comments