Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Improve ls output #187

Closed
mike0sv opened this issue Apr 4, 2022 · 6 comments
Closed

Improve ls output #187

mike0sv opened this issue Apr 4, 2022 · 6 comments
Assignees
Labels
cli MLEM command-line interface
Milestone

Comments

@mike0sv
Copy link
Contributor

mike0sv commented Apr 4, 2022

Now it is very simplistic and not informative. What info can we add?
Related to #153

@mike0sv mike0sv added the cli MLEM command-line interface label Apr 4, 2022
@mike0sv mike0sv added this to the Release Critical milestone Apr 4, 2022
@mike0sv
Copy link
Contributor Author

mike0sv commented Apr 8, 2022

@aguschin any thoughts?

@aguschin
Copy link
Contributor

@mike0sv
I would try to keep things simple in the first iteration:

  • name
  • is it a model/dataset
  • framework
  • size

Beside this we have requirements, input/output shapes, column names, method names, artifact uri (useful if meta/artifact are located in different places). Ideally, we should add those, but not sure it's useful, the lists could be quite long.
To print those I would use simpler command, something like pprint we had. But if we expose that command, its output should be more user-friendly than simple cat. So need to think how to do that (can colorize the output to make it easier to read at least).

@mike0sv
Copy link
Contributor Author

mike0sv commented Apr 14, 2022

I guess those are all related #168 #98

@mike0sv mike0sv self-assigned this Apr 14, 2022
@mike0sv
Copy link
Contributor Author

mike0sv commented Apr 14, 2022

also, need to catch errors on parsing objects and output them nicely

@aguschin
Copy link
Contributor

related to #279

@aguschin
Copy link
Contributor

aguschin commented Nov 9, 2022

ls was removed for now, closing this.

@aguschin aguschin closed this as completed Nov 9, 2022
@aguschin aguschin closed this as not planned Won't fix, can't repro, duplicate, stale Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cli MLEM command-line interface
Projects
None yet
Development

No branches or pull requests

2 participants