search

AgnostiqHQ / covalent

Pythonic tool for orchestrating machine-learning/high performance/quantum-computing workflows in heterogeneous compute environments.
https://www.covalent.xyz
Apache License 2.0
761 stars 91 forks source link

Include node id & executor label in dispatch graph #814

Closed mshkanth closed 2 years ago

mshkanth commented 2 years ago

Description To include the node id & executor label for each electron in the dispatch graph on the Web app GUI

Mandatory acceptance criteria User is able to view the node id & executor label of the electron along with each electron in the dispatch graph

image

Optional An option to show or hide the node id will help users. An option to show or hide the exector label will help users It will help to also show the node id & executor label as part of the electon side bar when an electron is selected

Implementation details

  1. Electron's node ID points to the transport_graph_node_id in the electrons table.
  2. Electron's dispatcher label is available in the dispatcher_name column starting from v11 of the database
mshkanth commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @ArunPsiog @Prasy12

mshkanth commented 2 years ago

@Prasy12 - FYI this is higher priority in the theme upgrade epic. Might have some BE dependency.

Prasy12 commented 2 years ago

@mshkanth Can you tell mw where exactly is the case for user viewing the node id on the UI? Is it available on Figma?

mshkanth commented 2 years ago

@mshkanth Can you tell mw where exactly is the case for user viewing the node id on the UI? Is it available on Figma?

Updated description with screenshot. Yes, it is there in Figma

mshkanth commented 2 years ago

@ArunPsiog @Prasy12 - Edited description with clarity on Db field for node Id

mshkanth commented 2 years ago

@Prasy12 - Updated issue to include executor label also.

mshkanth commented 2 years ago

Please add your planning poker estimate with ZenHub @Prasy12

Prasy12 commented 2 years ago

We have an option to hide labels in our current GUI now and till that is present we will show the executor and node id in a tooltip when the hide label feature is enabled. For normal view the UI will be as in the screenshot above : Without hide labels: image

With hide labels: image

cc : @santoshkumarradha @mshkanth @Kamalesh-suresh

Prasy12 commented 2 years ago

Hey @santoshkumarradha this is the implementation in the current application, please let me know if this is what you expect. If so we will create a PR for this after our QA by end of week. executorLabel_nodeid.webm