neural-api icon indicating copy to clipboard operation
neural-api copied to clipboard

add global variable to configure datasets directory

Open KoRiF opened this issue 4 years ago • 3 comments

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

`

KoRiF avatar Oct 26 '21 06:10 KoRiF

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

joaopauloschuler avatar Oct 27 '21 13:10 joaopauloschuler

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

KoRiF avatar Oct 27 '21 14:10 KoRiF

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

joaopauloschuler avatar Nov 18 '21 03:11 joaopauloschuler