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

feat: verdi process status --max-depth (#5568) #5727

Merged
merged 3 commits into from
Oct 30, 2022

Conversation

ltalirz
Copy link
Member

@ltalirz ltalirz commented Oct 28, 2022

AiiDA workflows can get very complex with many layers, leading verdi process status to print hundreds of lines of text.

The --max-depth allows the user to "zoom out" and look at the bigger picture, while still being able to "zoom in" on subworkflows of interest by calling verdi process status on those.

@ltalirz ltalirz linked an issue Oct 28, 2022 that may be closed by this pull request
@ltalirz ltalirz force-pushed the feat/verdi-process-status-max-depth branch from 03a98a7 to 389f7f1 Compare October 28, 2022 11:44
@ltalirz ltalirz marked this pull request as ready for review October 28, 2022 11:48
@ltalirz ltalirz force-pushed the feat/verdi-process-status-max-depth branch from 9970002 to 5b24b42 Compare October 28, 2022 12:38
AiiDA workflows can get very complex with many layers, leading `verdi
process status` to print hundreds of lines of text.

The `--max-depth` allows the user to "zoom out" and look at the bigger
picture, while still being able to "zoom in" on subworkflows of interest
by calling `verdi process status` on those.
@ltalirz ltalirz force-pushed the feat/verdi-process-status-max-depth branch from 7ece1eb to 57ba403 Compare October 28, 2022 13:21
@ltalirz ltalirz requested a review from sphuber October 28, 2022 22:54
@ltalirz
Copy link
Member Author

ltalirz commented Oct 28, 2022

this is a "cherry-pick" (+ conflict resolution) of a fix in 1.6

Copy link
Contributor

@sphuber sphuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ltalirz

@sphuber sphuber merged commit b575861 into aiidateam:main Oct 30, 2022
@sphuber sphuber deleted the feat/verdi-process-status-max-depth branch October 30, 2022 17:49
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.

add --depth option to verdi process status
2 participants