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

Skip questions in create script if files already exist #335

Closed
chrispcampbell opened this issue Jul 7, 2023 · 3 comments · Fixed by #562 or #563
Closed

Skip questions in create script if files already exist #335

chrispcampbell opened this issue Jul 7, 2023 · 3 comments · Fixed by #562 or #563
Assignees

Comments

@chrispcampbell
Copy link
Contributor

The create script currently asks some questions that are unnecessary if certain files already exist. For example:

  • It asks if you want to create a graph to get started, but we can skip this question if config/graphs.csv already exists.
  • It asks if you want to add an input variable (or variables) to get started, but we can skip this question if config/inputs.csv already exists.
  • It asks if you want to install the Emscripten SDK, but we can skip this question if emsdk is found in the directory hierarchy.
@chrispcampbell
Copy link
Contributor Author

This issue came up in discussion item #559. I'm changing this from an "improvement" to a "bug" because the first two items on the list above can cause confusion in setting up a new project if using a template that already includes a config folder (like the "sir" example does).

@chrispcampbell
Copy link
Contributor Author

I changed the "create" script to skip the questions that modify the graph and slider config files if there is a pre-existing config directory.

Without an existing config directory


With existing config directory

@chrispcampbell
Copy link
Contributor Author

chrispcampbell commented Nov 21, 2024

I also changed the "create" script to skip the question about installing the Emscripten SDK if there is a pre-existing emsdk directory in the hierarchy.

Without an existing emsdk directory


With existing emsdk directory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant