fix non-tuple indexing inside vigra

conda-forge / vigra-feedstock

A conda-smithy repository for vigra.

BSD 3-Clause "New" or "Revised" License

0 stars 14 forks source link

Closed k-dominik closed 2 years ago

k-dominik commented 2 years ago

make sure to index via tuples. numpy will error out since 1.23

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

k-dominik commented 2 years ago

@conda-forge-admin, please rerender

k-dominik commented 2 years ago

I'm currently reproducing the windows build... Let's see. Don't have access to a ~~non-mtl~~ non-silicon Mac today.

hmaarrfk commented 2 years ago

what is mtl? I've added a few more patches in attempts to solve this problem, but the cause seems to be elsewhere. https://github.com/conda-forge/vigra-feedstock/pull/97

k-dominik commented 2 years ago

what is mtl? I've added a few more patches in attempts to solve this problem, but the cause seems to be elsewhere. #97

heya, sorry I'm such a lazy typer. Since I'm still jumping between architectures I often give the silicon stuff a -mtl postfix...

hmaarrfk commented 2 years ago

why was this closed?