Disable OPENCV_SKIP_PYTHON_LOADER option to ensure that .pyi files are installed

[traversaro]

Hopefully this will fix #374 without creating additional problems.

Checklist

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

[traversaro]

@conda-forge-admin, please rerender

[hmaarrfk]

do you want to add a test?

[conda-forge-admin]

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/meta.yaml) and found it was in an excellent condition.

[conda-forge-admin]

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

_{This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11183329111.}

[hmaarrfk]

I wonder if it is finally time to drop qt5 and ffmpeg6

[traversaro]

do you want to add a test?

@Unturned3 @carschandler any idea for a reasonable test? I never used .pyi files, so I would just check the existence of the main .pyi in the root folder.

[traversaro]

I wonder if it is finally time to drop qt5 and ffmpeg6

Yes, I wanted to first check if there was any regression of this hence the full build, if I start iterating on it I will disable the builds.

[carschandler]

I think the main goal is to have LSP recognize the types but I’m not sure a great way to test that programmatically, so I think just checking the existence of the pyi files is good enough!

Also, it seems like this is on the right track as this is the PR that was just merged in nixpkgs to fix the same issue.

NixOS/nixpkgs#317546

[hmaarrfk]

Perhaps we can pull in the changes from #436 (if that PR works) into here and just merge.

Then get some feedback from users.

[traversaro]

Perhaps we can pull in the changes from #436 (if that PR works) into here and just merge.

Then get some feedback from users.

I am away from office for work all week, I can try do it but if anyone wants to do it before me, feel free to do it!

[hmaarrfk]

hmm i guess i closed the FFMPEG 6/7 dual builds. only ffmpeg7 now! whohooo!

[traversaro]

Thanks!

[github-actions]

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

• linter: passed
• azure: passed

Thus the PR was passing and merged! Have a great day!