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

Add config file ? wouldn't it be better than arg parsing ? #55

Closed
Clorr opened this issue Jan 5, 2018 · 2 comments
Closed

Add config file ? wouldn't it be better than arg parsing ? #55

Clorr opened this issue Jan 5, 2018 · 2 comments

Comments

@Clorr
Copy link
Contributor

Clorr commented Jan 5, 2018

Hi guys,

The more I think about it, the more I think complex arg parsing will be a problem.

Also if we want to move on to a GUI, it would be better to have a config file to set parameters durably.

We still can have an override for params through command line, so we can customize easily just the things we want. Something like ConfigArgParse does this for example.

What do you think about it?

@Ganonmaster
Copy link
Contributor

The command line is just another UI and again, you just use the command line arguments to fill out a config. I would rather vote for turning this project into a library and having separate projects for UI. That way you have a separate web UI, GUI and command line, but they all reference this same library.

@Yutsa
Copy link
Contributor

Yutsa commented Jan 5, 2018 via email

@Clorr Clorr closed this as completed Jan 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants