I know I wanted the DatasetLoader class to be stateless, but passing around the path is a chore and anyhow we create a new instance of a Loader every time we call handshape_datasets.load so there is not need to keep it stateless.
This would make it easier also to generate subpaths once in the constructor, and not repeat code in each functions (download, preprocess,load)
I know I wanted the DatasetLoader class to be stateless, but passing around the path is a chore and anyhow we create a new instance of a Loader every time we call handshape_datasets.load so there is not need to keep it stateless.
This would make it easier also to generate subpaths once in the constructor, and not repeat code in each functions (download, preprocess,load)