-
Notifications
You must be signed in to change notification settings - Fork 919
Update notebook samples for resource estimation visualization #816
Conversation
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 for updating the notebooks @katymccl. I left some comments.
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
@@ -380,6 +495,63 @@ | |||
"All other parameters remain unchanged." |
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.
Before the change, the code change to update the parameters came right after this explanation. Now there is a break in the flow with the time charts introduced before continuing with the updated parameters.
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.
yes this is a good point, i wanted to include the time and space diagrams in the results section but wasnt sure what to do with this section. I moved it to be under the time chart. Let me know what you think about that instead. It could also go right below the results summary table and then we have the space and time diagrams.
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
samples/azure-quantum/resource-estimation/estimation-dynamics.ipynb
Outdated
Show resolved
Hide resolved
…ipynb Co-authored-by: Mathias Soeken <mathias.soeken@microsoft.com>
…ipynb Co-authored-by: Mathias Soeken <mathias.soeken@microsoft.com>
…ipynb Co-authored-by: Mathias Soeken <mathias.soeken@microsoft.com>
…ipynb Co-authored-by: Mathias Soeken <mathias.soeken@microsoft.com>
updated the sample notebooks to include examples of showing the time and space diagrams to visualize resource estimation results.