Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addresses subscript out of bounds error #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Feb 28, 2024

  1. Addresses subscript out of bounds error

    I think I have found a fix for the error raised in piersyork#7. It's related to metadata which doesn't contain a source attribute.
    veggabo committed Feb 28, 2024
    Configuration menu
    Copy the full SHA
    950a816 View commit details
    Browse the repository at this point in the history

Commits on Feb 29, 2024

  1. changed proposed solution to the out of bound error

    instead of adding an if else statement, I moved the placement of assigning a value on the source attribute. If the source attribute is missing from the metadata, and it's the first value that's added to the `data_info` list, the rest of the list can't be populated, and it returns the "out of bound" error.
    veggabo committed Feb 29, 2024
    Configuration menu
    Copy the full SHA
    de13eb4 View commit details
    Browse the repository at this point in the history