• Mayank Mittal's avatar
    Fix the double ellipsis issue when resolving indices (#196) · c6af307a
    Mayank Mittal authored
    # Description
    
    Earlier in the code, we were using the `Ellipsis` object to index the
    dimensions of the tensor. This led to situations where
    we indexed multi-dimension tensors as: `x[..., ..., 0]`. This now leads
    to errors with Python 3.10.
    
    The MR replaces `Ellipsis` with the `slice(None)` object, which results
    in indexing as: `x[:, :, 0]`.
    
    ## Type of change
    
    - Bug fix (non-breaking change which fixes an issue)
    
    ## Checklist
    
    - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
    `./orbit.sh --format`
    - [ ] I have made corresponding changes to the documentation
    - [x] My changes generate no new warnings
    - [x] I have added tests that prove my fix is effective or that my
    feature works
    - [x] I have updated the changelog and the corresponding version in the
    extension's `config/extension.toml` file
    
    ---------
    Co-authored-by: 's avatarFarbod Farshidian <ffarshidian@theaiinstitute.com>
    c6af307a
Name
Last commit
Last update
.github Loading commit data...
.vscode Loading commit data...
docker Loading commit data...
docs Loading commit data...
source Loading commit data...
.dockerignore Loading commit data...
.flake8 Loading commit data...
.gitignore Loading commit data...
.pre-commit-config.yaml Loading commit data...
LICENSE Loading commit data...
README.md Loading commit data...
VERSION Loading commit data...
orbit.sh Loading commit data...
pyproject.toml Loading commit data...