The way I wrote it is a bit of a quick fix with a lot of code duplication, I'm sure it could be implemented in a cleaner way (e.g. having only one remove_files
method that can either take a single path or a list of paths).
It's one of those things that I intended to do at some point, but never had the time to clean it up (I did a similar modification for adding lists of files, since this has exactly the same issue if you don't want to add something you can define with a wildcard but only specific files).
If you're up for it, feel free - I'm sure there are plenty of people who would appreciate it.
Seems to be working. Is there any reason not to add this to the main repo?
Hi Allen,
I've ran into this exact problem myself, and simply added a function to dataset.py
in the clearml package ( clearml/datasets/dataset.py
) that takes a list of files instead of a single file.
It looks like this (I use clearml 1.13.1
):