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

con-duct inspect #173

Open
asmacdo opened this issue Sep 3, 2024 · 4 comments
Open

con-duct inspect #173

asmacdo opened this issue Sep 3, 2024 · 4 comments
Labels

Comments

@asmacdo
Copy link
Member

asmacdo commented Sep 3, 2024

I have data from an execution, and I have questions that I'd like to answer easily.

Heres what im thinking

con-duct info_filepath \
    --processes #  return the number of unique processes 
    --max-processes  #  max number of processes running concurrently
@yarikoptic
Copy link
Member

sorry -- what are you asking for , just general information from current instance, i.e. pretty much what would be stored in _info.json file? then I would vote for dump-info command. Potentially with some --format option if specific ones desired, and no jq in vicinity.

Note: I think we better use - within command name in CLI but underlying function sure thing would have _. Otherwise CLI would feel odd with _s in commands.

@asmacdo
Copy link
Member Author

asmacdo commented Sep 6, 2024

I was thinking this would be how to see any values that need to be calculated.

For example the total number of processes or the maximum number of simultaneous processes require reading over the entire usage.json

@yarikoptic
Copy link
Member

so the summary record of some kind out of collected stats -- cool. Again IMHO better to make it collect the record and provide capability to output as json (or may be even yaml) with some alternative rendering/filtering. Not unlike what we do in e.g. dandi-cli via pyout -- producing/updating display of records.

@asmacdo
Copy link
Member Author

asmacdo commented Sep 6, 2024

This feature isn't intended to replace that one, which currently is going to be called 'pretty print'

@asmacdo asmacdo added the suite label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants