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.

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

Add QCI simulator (Partner PR - Do Not Close) #608

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

Bradben
Copy link
Collaborator

@Bradben Bradben commented Jun 1, 2023

No description provided.

@prmerger-automator
Copy link
Contributor

Microsoft authors of articles need to make updates through the private repository, so the updates can be staged and validated by current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.

It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:

 Public repo URL: https://github.com/MicrosoftDocs/{docs-repo}
 Private repo URL: https://github.com/MicrosoftDocs/{docs-repo-pr}

We have closed this pull request. Please resubmit the updates through the private repo. For more information, see the contributor guide.

@Bradben Bradben reopened this Jun 1, 2023
@Bradben Bradben changed the title Add QCI simulator Add QCI simulator (Partner PR - Do Not Close) Jun 1, 2023
@learn-build-service-prod
Copy link

Learn Build status updates of commit c733c04:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/machines/backend-simulators.md ⚠️Warning Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

articles/machines/backend-simulators.md

  • Line 5, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '067/01/2023'. Must be a date in format M/D/YYYY, no more than 30 days from today.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit c2dcc28:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Bradben Bradben marked this pull request as draft June 1, 2023 23:15
@Court72
Copy link
Contributor

Court72 commented Jun 2, 2023

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Jun 2, 2023
@learn-build-service-prod
Copy link

Learn Build status updates of commit 74f4d93:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit 4824558:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@Bradben
Copy link
Collaborator Author

Bradben commented Jun 14, 2023

Thanks, @qci-amos - looks good to me.

@learn-build-service-prod
Copy link

Learn Build status updates of commit b857820:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link

Learn Build status updates of commit f39e653:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit f39e653b8263127513a21366626b91b0ad07b83d in branch qci-add of repository https://github.com/Bradben/quantum-docs into branch main (commit f3cafa1a661ad9eb2e4b1b0f303cc174e04a1681). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit 601f487:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link

Learn Build status updates of commit d05f33e:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@ffrachon
Copy link
Contributor

For this:
https://github.com/Bradben/quantum-docs/blob/qci-add/articles/hybrid-computing-integrated.md

Under
Integrated hybrid computing > Supported targets

Could we make this a single comparative table for all targets. It will make it easier for people to compare features between targets.

For this:
https://github.com/Bradben/quantum-docs/blob/qci-add/articles/machines/backend-simulators.md

  • "The QCI simulator currently supports 8 qubits in a ring topology and will let you simulate up to X shots."
    Do we know what is the value of X?

  • Also I would like the paragraphs starting with "Quantum Circuits, Inc. (QCI) is a full stack quantum computing" and "finishing with "to unlock new classes of quantum applications" to be written that does not use "We". Let's keep the same neutral, factual tone describing capabilities. It's a MS doc page that needs to keep the same style across all 3P hardware partners.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 11bea31:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 200a525:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod
Copy link

Learn Build status updates of commit a50493c:

✅ Validation status: passed

File Status Preview URL Details
articles/hybrid-computing-integrated.md ✅Succeeded
articles/machines/backend-simulators.md ✅Succeeded
articles/quantum-computing-target-profiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aq-pr-triaged tracking label for the PR review team do-not-merge needs-human-review
Projects
None yet
5 participants