I think that trainer class can be made more convenient and useful for more people. The most convenient feature that could be helpful in many applications is how hyperparameters are treated and automatic folders spawned with config.json and runs.jsonfiles. The issue is that Climate-Learning is too bulky and probably hard to understand for most people. To get more adopters simpler class would be preferred. This is not a easy thing to do because it depends heavily on other functions in Learn2_new.py of Climate-Learning/PLASIM. So here is a set of desired features and steps:
[ ] The class should be useful for anyone doing a parametric study and could be later inherited for downstream tasks
[ ] Check that similar class does not already exist somewhere on GitHub. Also consider writing it from scratch.
[ ] Trainer class stops being associated with a specific tasks of Climate-Learning: write a general class that has nothing to do with operations such as balancing, masking etc
[ ] Ideally it would not depend on whether the user chooses to work with tensorflow or pytorch
[ ] Make sure that the routines are as simple as possible. Basically try to reduce complexity of the code while prioritising utility
I think that trainer class can be made more convenient and useful for more people. The most convenient feature that could be helpful in many applications is how hyperparameters are treated and automatic folders spawned with
config.json
andruns.json
files. The issue is thatClimate-Learning
is too bulky and probably hard to understand for most people. To get more adopters simpler class would be preferred. This is not a easy thing to do because it depends heavily on other functions inLearn2_new.py
ofClimate-Learning/PLASIM
. So here is a set of desired features and steps:Climate-Learning
: write a general class that has nothing to do with operations such as balancing, masking etc