-
Notifications
You must be signed in to change notification settings - Fork 37
Add hello world samples #32
base: feature/samples-gallery
Are you sure you want to change the base?
Conversation
frtibble
left a comment
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.
Love these, they're a great addition - made a few minor suggestions (apologies for repeating, but figured it may be helpful). Happy to merge after that!
| "Optimization Algorithm | Target ID | Python Class\n", | ||
| "| --- | ---| ---| --- | -- |\n", | ||
| "[Tabu search](https://docs.microsoft.com/azure/quantum/provider-1qbit#tabu-search-solver) | `1qbit.tabu` | `azure.quantum.optimization.oneqbit.TabuSearch` |\n", | ||
| "[Path relinking](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `1qbit.pticm` | `azure.quantum.optimization.oneqbit.PathRelinkingSolver`\n", |
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.
Path relinking URL points to our solver - should be: "https://docs.microsoft.com/en-us/azure/quantum/provider-1qbit#path-relinking-solver" ?
| "Optimization Algorithm | Target ID | Python Class\n", | ||
| "| --- | ---| ---| --- | -- |\n", | ||
| "[Tabu search](https://docs.microsoft.com/azure/quantum/provider-1qbit#tabu-search-solver) | `1qbit.tabu` | `azure.quantum.optimization.oneqbit.TabuSearch` |\n", | ||
| "[Path relinking](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `1qbit.pticm` | `azure.quantum.optimization.oneqbit.PathRelinkingSolver`\n", |
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.
Table cell for path relinking says pticm but should be "1qbit.pathrelinking"?
| "| --- | ---| ---| --- | -- |\n", | ||
| "[Tabu search](https://docs.microsoft.com/azure/quantum/provider-1qbit#tabu-search-solver) | `1qbit.tabu` | `azure.quantum.optimization.oneqbit.TabuSearch` |\n", | ||
| "[Path relinking](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `1qbit.pticm` | `azure.quantum.optimization.oneqbit.PathRelinkingSolver`\n", | ||
| "[Parallel tempering with isoenergetic cluster moves](https://docs.microsoft.com/azure/quantum/provider-1qbit#pticm-solver) | `1qbit.pathrelinking` | `azure.quantum.optimization.oneqbit.PticmSolver`\n", |
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.
Table cell for parallel tempering says pathrelinking but should be "1qbit.pticm"?
| } | ||
| }, | ||
| "source": [ | ||
| "**See a solutiuon listed above? Congratulations, you've submitted an optimization job with Azure Quantum! 👏**" |
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.
typo: "solution"
| "| --- | ---| ---| --- | -- |\r\n", | ||
| "[Simulated annealing](https://docs.microsoft.com/azure/quantum/optimization-simulated-annealing) | `microsoft.simulatedannealing.cpu`,<br/>`microsoft.simulatedannealing-parameterfree.cpu`,<br/>`microsoft.simulatedannealing.fpga` | `azure.quantum.optimization.SimulatedAnnealing` |\r\n", | ||
| "[Parallel tempering](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `microsoft.paralleltempering.cpu`,<br/>`microsoft.paralleltempering-parameterfree.cpu`,<br/>`microsoft.paralleltempering.fpga` | `azure.quantum.optimization.ParallelTempering`\r\n", | ||
| "[Tabu search](https://docs.microsoft.com/en-us/azure/quantum/optimization-tabu-search) | `microsoft.tabu.cpu`,<br/>`microsoft.tabu-parameterfree.cpu` | `azure.quantum.optimization.Tabu`\r\n", |
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.
"azure.quantum.optimization.Tabu" can be removed
| "[Simulated annealing](https://docs.microsoft.com/azure/quantum/optimization-simulated-annealing) | `microsoft.simulatedannealing.cpu`,<br/>`microsoft.simulatedannealing-parameterfree.cpu`,<br/>`microsoft.simulatedannealing.fpga` | `azure.quantum.optimization.SimulatedAnnealing` |\r\n", | ||
| "[Parallel tempering](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `microsoft.paralleltempering.cpu`,<br/>`microsoft.paralleltempering-parameterfree.cpu`,<br/>`microsoft.paralleltempering.fpga` | `azure.quantum.optimization.ParallelTempering`\r\n", | ||
| "[Tabu search](https://docs.microsoft.com/en-us/azure/quantum/optimization-tabu-search) | `microsoft.tabu.cpu`,<br/>`microsoft.tabu-parameterfree.cpu` | `azure.quantum.optimization.Tabu`\r\n", | ||
| "[Quantum Monte Carlo](https://docs.microsoft.com/azure/quantum/optimization-quantum-monte-carlo) | `microsoft.qmc.cpu` | `azure.quantum.optimization.QuantumMonteCarlo`\r\n", |
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.
"azure.quantum.optimization.QuantumMonteCarlo" can be removed
| "[Parallel tempering](https://docs.microsoft.com/azure/quantum/optimization-parallel-tempering) | `microsoft.paralleltempering.cpu`,<br/>`microsoft.paralleltempering-parameterfree.cpu`,<br/>`microsoft.paralleltempering.fpga` | `azure.quantum.optimization.ParallelTempering`\r\n", | ||
| "[Tabu search](https://docs.microsoft.com/en-us/azure/quantum/optimization-tabu-search) | `microsoft.tabu.cpu`,<br/>`microsoft.tabu-parameterfree.cpu` | `azure.quantum.optimization.Tabu`\r\n", | ||
| "[Quantum Monte Carlo](https://docs.microsoft.com/azure/quantum/optimization-quantum-monte-carlo) | `microsoft.qmc.cpu` | `azure.quantum.optimization.QuantumMonteCarlo`\r\n", | ||
| "[Population annealing](https://docs.microsoft.com/azure/quantum/optimization-population-annealing) | `microsoft.populationannealing.cpu` | `azure.quantum.optimization.PopulationAnnealing`\r\n", |
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.
"azure.quantum.optimization.PopulationAnnealing" can be removed, and "microsoft.populationannealing-parameterfree.cpu" should be added
| "[Tabu search](https://docs.microsoft.com/en-us/azure/quantum/optimization-tabu-search) | `microsoft.tabu.cpu`,<br/>`microsoft.tabu-parameterfree.cpu` | `azure.quantum.optimization.Tabu`\r\n", | ||
| "[Quantum Monte Carlo](https://docs.microsoft.com/azure/quantum/optimization-quantum-monte-carlo) | `microsoft.qmc.cpu` | `azure.quantum.optimization.QuantumMonteCarlo`\r\n", | ||
| "[Population annealing](https://docs.microsoft.com/azure/quantum/optimization-population-annealing) | `microsoft.populationannealing.cpu` | `azure.quantum.optimization.PopulationAnnealing`\r\n", | ||
| "[Substochastic Monte Carlo](https://docs.microsoft.com/azure/quantum/optimization-substochastic-monte-carlo) | `microsoft.substochasticmontecarlo.cpu` | `azure.quantum.optimization.SubstochasticMonteCarlo`\r\n", |
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.
"azure.quantum.optimization.SubstochasticMonteCarlo" can be removed and "microsoft.substochasticmontecarlo-parameterfree.cpu" should be added
| { | ||
| "cell_type": "markdown", | ||
| "source": [ | ||
| "**See a solutiuon listed above? Congratulations, you've submitted an optimization job with Azure Quantum! 👏**" |
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.
typo: "solution"
Create hello world samples for notebooks sample gallery