Conversation
|
related to #1348 (comment) #1354 #1358 |
|
As mentioned here #1357 (comment), the current version has no API changes (except the type of dataset and collate_fn in However we don't actually need (But I find it hard to use |
| /// </summary> | ||
| /// <param name="index">Index for tensor</param> | ||
| /// <returns>Tensors of index. DataLoader will catenate these tensors into batches.</returns> | ||
| T this[long index] { get; } |
There was a problem hiding this comment.
Here we are using Int64. However actually most .net containers does not support Int64 indices. And actually it is impractical to have so much data. Should we switch to Int32 instead?
|
What is the status on this one? I waiting for this to be merged. |
|
Perhaps you could implement one yourself... This PR is one year old, and I'm not even sure if it conflicts with existing code... |
torch.utils.data.ConcatDatasetIDataset<out T>is addedDataset<T>implementsIDataset<T>DataLoader()has been added, to acceptIDatasetdatasetsDataLoader<T, S>, rather than its subclassesDataLoader<T, S>have been relaxed toIDataset<T>DataLoader<T, S>has been relaxed toIReadOnlyList