Refactor argparse - Githubissues

neuroevolution-ai / NeuroEvolution-CTRNN_new

MIT License

3 stars 3 forks source link

Refactor argparse #43

Closed bjuergens closed 3 years ago

bjuergens commented 3 years ago

reading of confs is actually unrelated: #20

This changes reading the arguments of train.py to use TAP instead of argparse.

Also some tweaks that go along with it.

bjuergens commented 3 years ago

Also I am thinking about doing some DRY-cleaning. Because train.py and render_hof.py both have some overlap because they both pass all their arguments to Experiment.

I'm thinking about creating a class ExperimentCLIArgs(Tap), which is used by both train,py and render_hof.py.

bjuergens commented 3 years ago

replaces argparse with tap
added option "--reset-hof"
option "--configuration" in train.py is now a positional argument
option "--dir" in render_hof.py is now a positional argument

can now be reviewed/merged