Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature enhancement and bug fixes #7

Open
wants to merge 156 commits into
base: main
Choose a base branch
from
Open

Feature enhancement and bug fixes #7

wants to merge 156 commits into from

Conversation

AndyZzzZzzZzz
Copy link

Summary of Updates:

  • Enabled local execution using MockDWaveSampler for testing and demonstration purposes.
  • Replaced embedding_helpers with the newly supported minorminer libraries for improved functionality.
  • Adjusted figure formatting to align with the style used in the published paper.
  • Implemented minor bug fixes and feature enhancements for better performance and usability.

Andy Zhang and others added 30 commits September 14, 2024 17:52
changed the version of numpy to 1.24.4 for the compatibility with dwa…
61e90fd 
…ve-gate 0.3.2
revert the version for numpy while specify the requirement for pydantic
91576f5
added pydanic version requirement
f2cb8e5
added mock sampler with topology pegasus
5ab1411
added mock sampler with topology pegasus
3464315
used chip_id for mock sampler's embeddings file name
06fa6fa
replace sampler.solver with sampler.properties[chip_id] in the save_e…
f8858d0 
…periment_data function
added mock sampler with pegasus topology, put constraint in loops for…
5d1a0c6 
… index out of bounds error as a temporary fix
WIP - need to resolve the index out of bounds error
e36cec0
Create a wrapper class for sampler
02eb30d
Adjust all examples to use wrapper class
3d6848c
Updated wrapper class to inherits from MockDWaveSampler
05e3734
Added argument description for sampler_type parameter
e9e74b4
Updated the script to take ShimmingSampler
64cb9e5
Override sample method in the wrapper class
45e63c9
Changed cached file path to use local version
2a653e0
Removed redundant imports
4ec0bf1
Added error checks and used Andrew's raster search
425d5c5
Updated example one with param[L] adjustment
60dbaf8
@jackraymond
Add suitably parameterized simulated annealing as default mocking sam…
6b2fe3d 
…pler
@jackraymond
Corrected special case handling of zero coupling, other tidy up
efbcbcb
Modified parameter to take a sampler and removed default to DWave Sam…
3517b5e 
…pler
Fixed the issue of embeddings, remove passing a graph to embed_loop f…
9554bc0 
…unction
Changed hard coded threshold for alpha_phi and alpha_J to parameters …
de6e956 
…in param
Adjust experiment parameters to get sensible results
81eb97f
Changed plots to be displayed on one canvas
efbf62f
Minor adjustment on format
70d06c5
Changed the parameters to accept a asmpler and uses raster embedding …
d945df3 
…search function from minorminer
Displayed plots on signle canvas
47de524
Changed parameters to match paper
bd1b659
Andy Zhang and others added 30 commits December 9, 2024 18:17
minor fix
cf0593c
@AndyZzzZzzZzz
Merge branch 'development' into andy
388dc67
Revert overriden changes in development
29160e4
Revert overriden changes in development
91146a7
Merge branch 'andy' of https://github.com/AndyZzzZzzZzz/shimming-tuto…
f4f468c 
…rial into andy
@AndyZzzZzzZzz
Merge pull request #4 from AndyZzzZzzZzz/andy
39cd247 
Used black to format code styling
Minor fix to matplotlib deprecated function
2810b53
Minor fix to matplotlib deprecated function
3624039
Removed embedding helperss
3286df8
Fixed pyflakes warnings
d16df56
Fixed pyflakes warnings
1309af3
Black reformat
bf33fc1
Black reformat
337bd19
@jackraymond
Merge pull request #5 from AndyZzzZzzZzz/andy
bab6378 
Minor fixes
Merge branch 'development' of https://github.com/AndyZzzZzzZzz/shimmi…
ae8aa59 
…ng-tutorial into development
Exposed max_num_emb in main parameter
308c2a1
Minor fix to the progress bar
4803cb4
Minor fix to max_num_emb parameter
0657f01
Added max_num_emb to arguments along with documentation
934a579
Added max_num_emb to arguments along with documentation
5632d21
Added max_num_emb to arguments along with documentation
3d6e782
Added default for max_num_emb in main and passed it into the function
a7a4304
Black reformat
fb4dc24
Merge remote-tracking branch 'upstream/main' into development
e8cae3a
Removed some duplicate statement introduced in merging
607b68a
Removed advantage property pkl file
3e435cc
@jackraymond
Change defaults for loop experiments back to max_num_embs=float('inf')
771159d
Match function name to most recent minorminer version
35c8061
Merge branch 'development' of https://github.com/AndyZzzZzzZzz/shimmi…
d921407 
…ng-tutorial into development
@jackraymond
Make consistent with minorminer new modules dwavesystems/minorminer#259…
3fa30bc 
…. Minor refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AndyZzzZzzZzz @jackraymond @kevinchern