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

Switch from workflow.dispatch to covalent.dispatch #48

Closed
kessler-frost opened this issue Jan 29, 2022 · 2 comments · Fixed by #55
Closed

Switch from workflow.dispatch to covalent.dispatch #48

kessler-frost opened this issue Jan 29, 2022 · 2 comments · Fixed by #55
Assignees
Labels
covalent-dispatcher Related to the dispatcher module documentation Improvements or additions to documentation feature New feature addition lattice Related to the lattice component

Comments

@kessler-frost
Copy link
Member

What should we add?

Currently we are using workflow.dispatch to send a lattice workflow for execution to the dispatcher server. Maybe we can use a more intuitive approach for dispatching which is similar to how we obtain the result for a particular dispatch. The proposal is to have a function covalent.dispatch similar to covalent.get_result so that it is more intuitive for users to send a job the same way they get the result.

Resolving this issue may also involve updating the README.mds, tutorials, documentation, etc. So it will turn into a big issue.

Describe alternatives you've considered.

No response

@kessler-frost kessler-frost added documentation Improvements or additions to documentation feature New feature addition covalent-dispatcher Related to the dispatcher module lattice Related to the lattice component labels Jan 29, 2022
@santoshkumarradha
Copy link
Member

santoshkumarradha commented Jan 30, 2022

Hey @kessler-frost , dont forget this list as well (could be separate issues as well)

  • including depreciation warning for lattice.dispatch()
  • Changing tutorials/how-to/readme/examples

Edit: already added

@santoshkumarradha
Copy link
Member

@kessler-frost another thing to remember is to move the non-executer but dispatcher dependent meta data in lattice like port etc.... inside this function. User should be able to dispatch the same lattice to separate dispatchers without needing to recreate the lattice.

AlejandroEsquivel pushed a commit that referenced this issue Aug 16, 2023
…et-executors

Change to Braket executors dict method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
covalent-dispatcher Related to the dispatcher module documentation Improvements or additions to documentation feature New feature addition lattice Related to the lattice component
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants