Fabrizio Ottati

Results 51 comments of Fabrizio Ottati

> on your machine you can obviously have loris installed. It's meant as an optional dependency, so that Tonic works also when it's not installed, provided you don't use certain...

I think it is a good idea! Then, we should do this for all the datasets in which `io.read_aedat4()` is used, since that function depends on `loris`, right?

There is [CIFAR10](https://github.com/neuromorphs/tonic/blob/bc9fd1317a709c291bc0f16881d2ee7de2a2fcb6/tonic/datasets/cifar10dvs.py#L4) that depends on `loris`, for instance. The hd5py option seems fine! Will do!

I would like to point out, given the PR on `make_structured_array`, that in CIFAR10DVS, at [this line](https://github.com/neuromorphs/tonic/blob/bc9fd1317a709c291bc0f16881d2ee7de2a2fcb6/tonic/datasets/cifar10dvs.py#L91), an error is returned because `loris` already provides a structured array. In particular,...

Hi @biphasic. I have packaged both NCARS and CIFAR10DVS in HDF5 format, as you suggested, removing the `loris` dependency. Where can I upload these files?

Don't worry @biphasic. I googled a little bit and I found out that it is possible to upload the dataset on Zenodo, which is pretty reliable. I was wondering about...

@biphasic I totally agree with you on both points. Let's see how it goes with them. Meanwhile, CIFAR10 and NCARS can be used with loris as an external dependency, so...

Hi! I saw that you mentioned that NCARS is already available in Tonic but I am not able to find it. Is it hidden in some other datasets? Or it...

Yes, I need to work with the dataset from the original HATS paper. Do you suggest to manually download NCARS from their website and use loris to handle it? Of...

I noticed something: on [this line](https://github.com/neuromorphs/tonic/blob/bc9fd1317a709c291bc0f16881d2ee7de2a2fcb6/tonic/io.py#L25), loris is used directly without the try/except block. In fact, `read_aedat4()` is used also in [CIFAR10DVS](https://github.com/neuromorphs/tonic/blob/bc9fd1317a709c291bc0f16881d2ee7de2a2fcb6/tonic/datasets/cifar10dvs.py#L90) to read the file. Hence, we can simply...