search

joaopauloschuler / neural-api

CAI NEURAL API - Pascal based deep learning neural network API optimized for AVX, AVX2 and AVX512 instruction sets plus OpenCL capable devices including AMD, Intel and NVIDIA.
GNU Lesser General Public License v2.1
356 stars 195 forks source link

add global variable to configure datasets directory #72

Open KoRiF opened 2 years ago

KoRiF commented 2 years ago

this allows to load CIFAR10 & CIFAR100 datasets from the arbitrary location

by default, directory initialized as empty string, so this should not affects on the existing logic

usage:

` ... uses neuraldatasets; //usual import //initialization neuraldatasets.directory := '..\..\Data\cifar-10-batches-bin\'; // path to CIFAR-10* //including trailing path delimiter

`

joaopauloschuler commented 2 years ago

The main idea is good. It's against my religion to have global variables. So, I'll implement your solution passing parameters.

KoRiF commented 2 years ago

in the place where the dataset is loaded, the religion is procedural programming style

in a situation where there are no classes, using a global variable is not a religious crime.

I mean, the module is called "neural dataset", and if it were a class, then the directory with the dataset could well be its field

i.e. this global variable really defines the context, and is not a parameter of each new call to the algorithm

therefore, probably, your gods would forgive us this sin

2021-10-27 16:32 GMT+03:00, joaopauloschuler @.***>:

The main idea is good. It's against my religion to have global variables. So, I'll implement your solution passing parameters.

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/joaopauloschuler/neural-api/pull/72#issuecomment-952933712

joaopauloschuler commented 2 years ago

The request is valid and I'm still considering it.