Skip to content

Commit

Permalink
feat: det (notebook|tensorboard|shell|command) ls --sort-by ... [DE…
Browse files Browse the repository at this point in the history
…T-6126] (#8649)

- Add frequently requested option to sort NTSC output. Sort by startTime
  by default.
- There were some inconsistencies in the `args_description` typing and
  formatting. These are now rectified.
- Rename `cli/command.py` -> `cli/ntsc.py` and `cli/remote.py -> cli/command.py`
  to better reflect their contents.
  • Loading branch information
ioga authored Jan 10, 2024
1 parent 39ad129 commit 0d93855
Show file tree
Hide file tree
Showing 13 changed files with 648 additions and 636 deletions.
2 changes: 2 additions & 0 deletions .git-blame-ignore-revs
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,5 @@
85396ebc4f0ec95c6b4ef9625061c92fa324cdce
# chore: migrate shared-web to subtree (#4402)
0505422648e7f98aa25495b4cc525cd776a87840
# style: reformat `det (n|t|s|c)` arguments code. (#8652)
7d9f29532492788b05051e6745234cdaf6a0fa4e
2 changes: 1 addition & 1 deletion harness/.flake8
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,11 @@ per-file-ignores =
determined/cli/master.py:I2041
determined/cli/model.py:I2041
determined/cli/notebook.py:I2041
determined/cli/ntsc.py:I2041
determined/cli/oauth.py:I2041
determined/cli/project.py:I2041
determined/cli/proxy.py:I2041
determined/cli/rbac.py:I2041
determined/cli/remote.py:I2041
determined/cli/render.py:I2041
determined/cli/resource_pool.py:I2041
determined/cli/resources.py:I2041
Expand Down
2 changes: 1 addition & 1 deletion harness/determined/cli/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,14 @@
agent,
checkpoint,
cli,
ntsc,
command,
experiment,
master,
model,
notebook,
project,
rbac,
remote,
render,
resources,
shell,
Expand Down
4 changes: 2 additions & 2 deletions harness/determined/cli/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
from determined.cli import render
from determined.cli.agent import args_description as agent_args_description
from determined.cli.checkpoint import args_description as checkpoint_args_description
from determined.cli.command import args_description as command_args_description
from determined.cli.dev import args_description as dev_args_description
from determined.cli.experiment import args_description as experiment_args_description
from determined.cli.job import args_description as job_args_description
Expand All @@ -33,7 +34,6 @@
from determined.cli.oauth import args_description as oauth_args_description
from determined.cli.project import args_description as project_args_description
from determined.cli.rbac import args_description as rbac_args_description
from determined.cli.remote import args_description as remote_args_description
from determined.cli.resource_pool import args_description as resource_pool_args_description
from determined.cli.resources import args_description as resources_args_description
from determined.cli.shell import args_description as shell_args_description
Expand Down Expand Up @@ -169,7 +169,7 @@ def render_sequence(sequence: List[str]) -> str:
+ template_args_description
+ tensorboard_args_description
+ trial_args_description
+ remote_args_description
+ command_args_description
+ user_args_description
+ user_groups_args_description
+ rbac_args_description
Expand Down
Loading

0 comments on commit 0d93855

Please sign in to comment.