-
Notifications
You must be signed in to change notification settings - Fork 15
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
Comments
@giovannipizzi @cpignedoli too simple? |
I like it! Maybe we can hide the initial magnetic moments if unset, or write something so say that they were not set |
@giovannipizzi @cpignedoli regarding the simplified process tree... humanized_process_tree.mp4Feedback please 🙏 Note that one issue already is that the total jobs is currently updateI 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 |
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., Only typographical note, I would write |
@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. |
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 forLast modification time
.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)
The text was updated successfully, but these errors were encountered: