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

Job submission #1479

Merged
merged 8 commits into from
Jul 12, 2022
Merged

Conversation

jmonas
Copy link

@jmonas jmonas commented Jul 7, 2022

job submission, listing, and pulling of results

@jmonas jmonas requested a review from ricardo-espinoza July 7, 2022 23:35
@ghost
Copy link

ghost commented Jul 7, 2022

CLA assistant check
All CLA requirements met.

global.json Outdated
@@ -1,6 +1,6 @@
{
"sdk": {
"version": "6.0.100",
"version": "6.0.300",
Copy link
Contributor

Choose a reason for hiding this comment

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

We're in the process of taking this change in another change. Do you need 6.0.300 in order for the changes to work? In any case, you should exclude this from the PR, even if locally you're using 6.0.300 for testing.

Copy link
Author

Choose a reason for hiding this comment

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

I forgot to change back when troubleshooting job submission issues. Changes will work with 6.0.100

@@ -1,6 +1,6 @@
{
"name": "quantum-devkit-vscode",
"version": "#VSCODEVSIX_VERSION#",
"version": "0.9999.22060152",
Copy link
Contributor

Choose a reason for hiding this comment

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

Make sure that your v.template file keeps the #VSCODEVSIX_VERSION#. We can chat offline if you have questions about how this works.

Copy link
Author

Choose a reason for hiding this comment

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

Gotcha, thanks would be great to hear about how it works.

@@ -74,7 +119,7 @@
"null"
],
"description": "Path to the language server assembly for Q#.",
"default": null
"default": "C:/Users/t-jackmonas/work/qsharp-compiler/src/QsCompiler/LanguageServer/bin/Debug/net6.0/win10-x64/publish/Microsoft.Quantum.QsLanguageServer.exe"
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't include this in the PR. It makes sense to change locally while you're developing, but it will help us avoid an accident if this never makes it to the repo.

@@ -0,0 +1,3 @@
{
"optimizationProviders":["Microsoft"]
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be better to exclude targets than providers, since Microsoft could later on add a QC target, or another provider could add an optimization one.

@ricardo-espinoza
Copy link
Contributor

Let's make sure we get an official icon for the graphic used in this PR.

For now, we can definitely use the one you included, but before the project is over, we could request one to be provided to us.

@jmonas jmonas merged commit cd422fb into microsoft:t-jackmonas/dev/jobSubmission Jul 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants