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

Improve results tab to make it more user-friendly #1042

Open
5 tasks done
edan-bainglass opened this issue Dec 28, 2024 · 5 comments
Open
5 tasks done

Improve results tab to make it more user-friendly #1042

edan-bainglass opened this issue Dec 28, 2024 · 5 comments
Assignees

Comments

@edan-bainglass
Copy link
Member

edan-bainglass commented Dec 28, 2024

  • Make the tree view hidden by default, e.g. in a tab "Advanced workflow tree view"
  • Replace with simpler tickbox interface showing what is not run yet and waiting for dependencies, what is running (orange emoji?) and what is finished, with a quick overview as well as clickable links to the relevant calculations. See suggestions here: Image
  • In the "Main settings", add information on the structure (Chemical formula, number of atoms, maybe even space group and cell vectors (a, b, c, alpha, beta, gamma)
  • In the title "QE App workflow (pk: 909)" replace instead simply with: Workflow details and, among the very first settings (or even better, before "Main settings", a first category "Workflow properties", with the following bullet points: PK, UUID, Label, Description, Creation time: 12 May 2024 12:42 (3 months ago) (with the suggested format before), and same for Last modification time.
  • In the advanced settings, write something near "Initial magnetic moments" if not set (e.g. "non-magnetic calculation" or "No initial moments set") - hidden if unset

Remember not to use any AiiDA language in the simple view (OK to use "Workflows" or "Calculations" or similar general terms, ideally defining internally what we mean with them so we use them consistently, but no WorkChain or PhononWorkChain or similar)

@edan-bainglass
Copy link
Member Author

@giovannipizzi @cpignedoli too simple?

Image

@giovannipizzi
Copy link
Member

I like it! Maybe we can hide the initial magnetic moments if unset, or write something so say that they were not set

@edan-bainglass
Copy link
Member Author

edan-bainglass commented Dec 31, 2024

@giovannipizzi @cpignedoli regarding the simplified process tree...

humanized_process_tree.mp4

Feedback please 🙏

Note that one issue already is that the total jobs is currently len(node.called), but that grows dynamically as you can see. I need some way to know in advance what jobs are planned to run. Thinking... input welcomed!

update

I can extract the counts from root.get_metadata_inputs() 👍 This works quite nicely, though it does require a bit of logic to handle certain exceptions.

humanized_process_tree.mp4

@giovannipizzi
Copy link
Member

Looks good! If you manage to fix the number of total steps that's great, maybe keep the logic confined in a function if possible, with good docs. Otherwise ok for me to have a fallback mechanism where the total number might change (e.g., I don't think we can know in advance how many restarts the relax worckchain or scf workchain will do). Rsync does the same, one can also wrote it in some doc page or as a note, or use a symbol to say that the number is estimated (e.g., 3/5*), or even don't write at all the total when it's unknown.

Only typographical note, I would write k-points with k always lower case. Also, personally I'd use the dash only if it's an adjective, so "k-point density" but "number of k points", but this is probably a bit more debatable

@edan-bainglass
Copy link
Member Author

edan-bainglass commented Jan 5, 2025

@giovannipizzi see #1048 for current status. The total number is resolved, though I did not consider restart workflows. I can make a special case for them, in which it will take on a dynamic form (the * is a nice added touch).

As for k-points, I agree that k should be lower case. You can probably tell my opinion regarding the dash from the previous sentence 😅 Though I think from an English perspective, you are correct.

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

No branches or pull requests

2 participants