-
Notifications
You must be signed in to change notification settings - Fork 35
Add MCP #689
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
Open
Jan Unsleber (nabbelbabbel)
wants to merge
46
commits into
main
Choose a base branch
from
jpu/mcp
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Add MCP #689
Changes from 42 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
72bb2aa
Add remote execution and QDK Chemistry tooling
2efc756
post merge cleaning part 1
ac8e392
post merge cleaning part 2
01064cd
Potential fix for pull request finding 'Unused local variable'
nabbelbabbel 12dbaa6
Potential fix for pull request finding 'Unused local variable'
nabbelbabbel 9cc2204
Merge remote-tracking branch 'origin/main' into jpu/mcp
c7dd998
fix test
76563fd
Add missing argcomplete dependency
300d9cf
Fixes
27b8f81
remove old code
7887228
cleaning
1d1dba3
rework tests
57ddb35
Potential fix for pull request finding
nabbelbabbel 55f6840
Potential fix for pull request finding
nabbelbabbel 4b24bdc
Potential fix for pull request finding
nabbelbabbel 37902df
Merge remote-tracking branch 'origin/main' into jpu/mcp
9236b6c
docs: narrow MCP workflow guidance
d1f7276
Add MCP-safe remote backend configuration
ec28e11
Harden MCP project and remote job isolation
4424138
Harden MCP UI tools and remote job handling
79f4fac
resolve remaining comments
cf4a301
Potential fix for pull request finding 'Unused local variable'
nabbelbabbel ccd22d1
fixes
nabbelbabbel 9ef4254
fixes
nabbelbabbel df520fe
cleaning
nabbelbabbel 53ccbec
fixes and comment responses
nabbelbabbel a03527b
align and check versions in skills
nabbelbabbel d45a10c
clean alias handling
nabbelbabbel 37910f8
Fixes
nabbelbabbel 25e0ad6
Docs
nabbelbabbel 10a0f81
Add circuit estimation MCP support and algorithm hashing
nabbelbabbel 6399f79
Merge remote-tracking branch 'origin/main' into jpu/mcp
nabbelbabbel 7245f31
fixes for arm64, make mcp a python plugin, rework directory handling …
nabbelbabbel 12ab8b7
fixes
nabbelbabbel 672dc85
Move nuclear derivative changes to dedicated branch
nabbelbabbel 4d76468
fixes
nabbelbabbel 5dea9bc
revert bad changes
nabbelbabbel 9038729
fixes
nabbelbabbel 79386df
fix
nabbelbabbel f8a82d2
Merge branch 'main' into jpu/mcp
nabbelbabbel e73c775
fix test issue
nabbelbabbel 7b58d10
Potential fix for pull request finding
nabbelbabbel 58586bd
Potential fix for pull request finding
nabbelbabbel f6b184c
Merge remote-tracking branch 'refs/remotes/origin/jpu/mcp' into jpu/mcp
nabbelbabbel 876ec7d
resolve comment tentative test fix
nabbelbabbel ef5f07e
fix
nabbelbabbel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| { | ||
| "name": "qdk-chemistry", | ||
| "owner": { | ||
| "name": "Microsoft" | ||
| }, | ||
| "metadata": { | ||
| "description": "QDK Chemistry agent plugins", | ||
| "version": "2.1.0" | ||
| }, | ||
| "plugins": [ | ||
| { | ||
| "name": "qdk-chemistry", | ||
| "description": "Run QDK Chemistry workflows with specialized agents, skills, and MCP tools.", | ||
| "version": "2.1.0", | ||
| "source": "copilot-plugins/qdk-chemistry" | ||
| } | ||
| ] | ||
| } | ||
|
Copilot marked this conversation as resolved.
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,6 +48,7 @@ pip-delete-this-directory.txt | |
|
|
||
| # Unit test / coverage reports | ||
| htmlcov/ | ||
|
|
||
| .tox/ | ||
| .nox/ | ||
| .coverage | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| { | ||
| "mcpServers": { | ||
| "qdk_chemistry": { | ||
| "type": "stdio", | ||
| "command": "qcmcp", | ||
| "timeout": 1814400000, | ||
| "env": { | ||
| "QDK_REQUIRE_WORKSPACE_BINDING": "1" | ||
| } | ||
| } | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| { | ||
| "name": "qdk-chemistry", | ||
| "description": "Use QDK Chemistry through skills and MCP tools.", | ||
| "version": "2.1.0", | ||
| "author": { | ||
| "name": "Microsoft" | ||
| }, | ||
| "skills": "skills/", | ||
| "mcpServers": ".mcp.json" | ||
| } |
65 changes: 65 additions & 0 deletions
65
copilot-plugins/qdk-chemistry/skills/qdk-chemistry-coding/SKILL.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,65 @@ | ||
| --- | ||
| name: qdk-chemistry-coding | ||
| version: 'v2.1.0' | ||
| description: 'Describes the Python API for discovering, configuring, and executing QDK Chemistry algorithms.' | ||
| --- | ||
|
|
||
| # QDK Chemistry Python API | ||
|
|
||
| Algorithms are obtained from the registry: | ||
|
|
||
| - `available(algorithm_type)` returns registered implementations. | ||
| - `show_default(algorithm_type)` returns the registered default. | ||
| - `create(algorithm_type, algorithm_name, **settings)` creates an algorithm. | ||
| - `algorithm.settings()` returns its settings object. | ||
| - `algorithm.run(...)` executes it. | ||
| - `algorithm.hash(...)` computes the content hash for a call. | ||
| - `algorithm.run(..., remote=name_or_backend)` executes it through a registered | ||
| remote backend and waits for the result. | ||
|
|
||
| Algorithm `run` signatures and data-class APIs define accepted inputs and | ||
| returned objects. Coordinates supplied to `Structure` are in Bohr. The | ||
| registry defines installed algorithm types, implementation names, settings, | ||
| and defaults. | ||
|
|
||
| ## Settings | ||
|
|
||
| `settings()` returns typed setting entries. `get(name)` reads a value and | ||
| `set(name, value)` changes it before execution. Algorithm references encode a | ||
| nested algorithm selection and its nested settings. Validation errors identify | ||
| unknown setting names or incompatible values. | ||
|
|
||
| ## Execution and Results | ||
|
|
||
| Each algorithm class defines its own `run(...)` signature. Inputs include data | ||
| objects and scalar configuration such as charge or multiplicity where required. | ||
| Returns may be a data object, a scalar, or a tuple. The API documentation for | ||
| the selected class defines the exact return shape. | ||
|
|
||
| Data objects support JSON or HDF5 serialization according to their class API. | ||
| Serialized files can be loaded and passed to later calls that accept the same | ||
| data type. | ||
|
|
||
| ## Content Hashes | ||
|
|
||
| Algorithms and data objects expose deterministic content hashes. An algorithm | ||
| call hash includes the implementation, settings, and inputs. Cache backends use | ||
| that identity to associate a call with serialized outputs. | ||
|
|
||
| ## Remote Execution | ||
|
|
||
| `algorithm.run(..., remote=name_or_backend)` routes a call through a registered | ||
| backend and returns the same result shape as local execution. The module-level | ||
| `qdk_chemistry.remote.run(algorithm, ..., remote=name_or_backend)` function | ||
| provides the same blocking behavior for any compatible algorithm object. | ||
|
|
||
| For asynchronous execution, | ||
| `qdk_chemistry.remote.submit(algorithm, ..., remote=name_or_backend)` returns a | ||
| `Job`. A job can be checked, waited on, fetched, canceled, saved, and loaded. | ||
| Pass `job_dir` to `submit` to save its record automatically; otherwise the job | ||
| is returned in memory. | ||
|
|
||
| Use `available_backends()` to inspect registered SDK backends and | ||
| `create_remote(name, **configuration)` to create a connected backend instance. | ||
| The package ships only the `local` subprocess backend. Installed plugins can | ||
| register additional backends. |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.