Ariadna
Ariadna
> I don't know which is the need for a new class. I think that the same can be achieved by templating these methods in the base class and specializing...
> Good work! I would suggest to move the implementation to the correponding cpp file to have less compile time dependency between files and minimize the compilation time. Totally agree...
Ping @RiccardoRossi @rubenzorrilla
> Good work! I would suggest to move the implementation to the correponding cpp file to have less compile time dependency between files and minimize the compilation time. I tried...
> > > > > > > > > I tend to agree @pooyan-dadvand about this. Also note that this is the general rule of the project (we use header-only...
> > > > > > > > > > > > > > > > > I tend to agree @pooyan-dadvand about this. Also note that this is the...
The implementation could be done in different ways * As a list of elements "@include_json" : ["file_name1.json", file_name2.json"] * Unique element "@include_json" : "file_name.json" This case is more simple but...
> > Besides, I think it would be a good idea to rename the class to `VoxelUtilities` so we can wrap up all the future implementations in here (not only...