williamFalcon / test-tube

Python library to easily log experiments and parallelize hyperparameter search for neural networks
MIT License
735 stars 74 forks source link

Write basic tests for HyperOptArgumentParser #18

Open williamFalcon opened 6 years ago

williamFalcon commented 6 years ago

Initial tests should include:

yuqli commented 5 years ago

I claimed to contribute to the "writing tests for log.py issue" a few weeks ago but just got started, and soon realize it might be helpful to understand argparse_hopt.py first as the Experiment class depends on it.

I have been trying to read the source code for the past two days and sort of understand the Experiment class now, but it would be great if you could kindly help offer some comments on the general purpose of the following classes in the argparse_hopt.py script... I'm not sure I entirely got what's happening in this file.

Also, is hyperopt.py obsolete? It seems not being used anywhere or I might be missing something.

Thanks a lot!

williamFalcon commented 5 years ago

hyperopt.py is deprecated. TTNamespace is a weird python object for holding key-value pairs. OptArg has logic for manipulating a parameter HyperOptArgumentParser not sure what you're asking. This class is there for convenience to the user.