Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Update RE application samples to use new features in Q# and azure_quantum #811

Merged
merged 7 commits into from
Jul 11, 2023
Merged

Update RE application samples to use new features in Q# and azure_quantum #811

merged 7 commits into from
Jul 11, 2023

Conversation

msoeken
Copy link
Member

@msoeken msoeken commented Jun 29, 2023 •
edited
Loading

This sample uses

  • constraints.logical_depth_factor for the Dynamics sample (with an explanation)
  • Q# input arguments for the Dynamics sample
  • AccountForEstimates in the Factoring sample
  • summary_data_frame for all samples

@msoeken msoeken marked this pull request as draft June 29, 2023 15:46
@msoeken msoeken changed the title Use AccountForEstimates in Factoring sample Update RE application samples to use new features in Q# and azure_quantum Jun 29, 2023
@msoeken msoeken marked this pull request as ready for review June 30, 2023 09:27
DmitryVasilevsky
DmitryVasilevsky approved these changes Jul 11, 2023
View reviewed changes
Copy link
Contributor

@DmitryVasilevsky DmitryVasilevsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving 'AccountForEstimates' part with a minor comment.

@msoeken msoeken merged commit b7dab2d into main Jul 11, 2023
12 checks passed
@msoeken msoeken deleted the msoeken/factoring-sample branch July 11, 2023 16:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DmitryVasilevsky DmitryVasilevsky DmitryVasilevsky approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@msoeken @DmitryVasilevsky