• 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
..
CHANGELOG.rst Loading commit data...
README.md Loading commit data...