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

Update CLIs to use "review" base command #39

Open
WilliamsTravis opened this issue May 17, 2023 · 3 comments
Open

Update CLIs to use "review" base command #39

WilliamsTravis opened this issue May 17, 2023 · 3 comments

Comments

@WilliamsTravis
Copy link
Collaborator

WilliamsTravis commented May 17, 2023

Why this feature is necessary:
This syntax will make it easier to find available review CLI commands. For our reVX CLI, we have a lot of commands that are not intuitively associated with reVX (e.g., setbacks) while for the reV model we have a reV base command to give the user a nice place to start. I find the latter much more intuitive and easy to work with.

A possible solution is:
Use a generic review CLI with command options. So, you would type review and then the help file would list the available commands. The option would then run the target command, so, something like this:

review make-maps -i input.csv -t wind -o wind_maps/ -d 300

@WilliamsTravis
Copy link
Collaborator Author

We already have a reView command that launches the development server for the DASH gui. This would have to become something like review gui.

@mjgleason
Copy link
Collaborator

How about reView app for the DASH gui?

@WilliamsTravis
Copy link
Collaborator Author

review app, let's not do caps.

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

No branches or pull requests

2 participants