Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Improve CLI help prompt in terminal #166

Merged
merged 2 commits into from
Apr 13, 2021
Merged

Improve CLI help prompt in terminal #166

merged 2 commits into from
Apr 13, 2021

Conversation

adason
Copy link
Contributor

@adason adason commented Apr 7, 2021 •
edited
Loading

Peer Review Information

  • Updated CLI help prompt and improved documentation
  • Removed datasetinsights.command in the sphinx docs. We don't expect users to import command module as this is not part of the public API.
  • Added visualization documentation in sphonx

before:
Screen Shot 2021-04-07 at 4 12 57 PM
after:
Screen Shot 2021-04-07 at 4 12 24 PM

Pull Request Check List

  • Added tests for changed code.
  • Updated documentation for changed code.

@adason adason requested review from 86sanj and salehe-ee April 7, 2021 23:18
salehe-ee
salehe-ee approved these changes Apr 9, 2021
View reviewed changes
Copy link
Contributor

@salehe-ee salehe-ee left a comment

Choose a reason for hiding this comment

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

Download from Unity Simulation works as expected. The improved CLI help prompt looks good!

@adason adason merged commit 3b6529a into master Apr 13, 2021
@adason adason deleted the fix_cli branch April 13, 2021 00:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@salehe-ee salehe-ee salehe-ee approved these changes

@86sanj 86sanj 86sanj approved these changes

Assignees

@86sanj 86sanj

@salehe-ee salehe-ee

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adason @86sanj @salehe-ee