activeloopai / activeloopai/deeplake

[Improvement] Make index errors while accessing Dataset and Tensor easier to understand.

Open
#1,560 9 comments 0 reactions 1 assignee Claimed by @Moksh45 View on GitHub
Dominant language
C++
Stars
9.2k
Forks
722
PR merge metrics
No merged PRs in 30d

Description

## 🚨🚨 Feature Request

- [x] A new implementation (Improvement, Extension)

### Is your feature request related to a problem?

Currently, if a user tries to access an index that is larger than the dataset length or tensor length, an internal error is thrown which is not easy to understand.

### Description of the possible solution

We can catch the error and throw a more descriptive error that's easier to understand.

Example of when the error is thrown:-

```python
ds = hub.dataset("path/to/dataset")
ds.create_tensor("xyz")
for i in range(10):
ds.xyz.append(i)

# now length of xyz and ds is 10

ds[20] # throws error
ds.xyz[89] # throws error
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.