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

NetQASM interfaces for QNE/QNOS integration #63

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

cmvanb
Copy link

@cmvanb cmvanb commented Nov 28, 2023

Interfaces for NetQASM programs intended to run in multiple environments (e.g. SquidASM and QNodeOS) with identical code.

@cmvanb cmvanb marked this pull request as ready for review January 26, 2024 15:11
@cmvanb cmvanb force-pushed the qne-qnos-integration branch from ebb231e to 9cbd7f6 Compare January 26, 2024 16:45
@cmvanb cmvanb removed the request for review from wkozlowski-tudelft February 1, 2024 14:28
@cmvanb cmvanb force-pushed the qne-qnos-integration branch from 45ed998 to 0cc4f97 Compare February 2, 2024 10:14
@bvdvecht
Copy link
Collaborator

Hi Casper, thanks for your PR!

The interface seems simple enough and therefore I see no issues with it.

I do have some more general questions regarding the intention of this interface:

  • How does this relate to the Application class in netqasm.runtime.application?
  • How will programs be instantiated and executed? Will this also be defined in this interface in the future?
  • Is the intention to port current application source code in this repository to use this new interface?
  • Is the interface intended to be used only with the "multithreading" variant of SquidASM? The "stack" variant requires the programmer to put yield keywords in the source code, which makes the application code not be exactly the same as one would write for say, QNodeOS.

Looking forward to hear your thoughts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants