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

Test warnings with setuptools-based get_info #92

Closed
wants to merge 4 commits into from

Conversation

maresb
Copy link
Contributor

@maresb maresb commented Jul 12, 2022

Check for warnings as per aesara-devs/aesara#980 (comment)

The warnings are already in allowed-warnings-*.txt, so we have to check the Windows CI logs.

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.

@conda-forge-linter
Copy link

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.

@brandonwillard
Copy link
Member

brandonwillard commented Jul 12, 2022

I didn't see any WARN: * messages in the Windows CI logs I looked at, so perhaps that issue is solved; however, one of the Windows tests failed, but I'm not sure about exactly what that failure is indicating.

@maresb
Copy link
Contributor Author

maresb commented Jul 13, 2022

It does seem to me like the issue has been solved! 😅

But in order to properly check for warnings regarding the loading of BLAS, it looks like I need to also import aesara.tensor which imports aesara.tensor.blas which imports aesara.tensor.blas_headers which on pypy 3.7 on Windows in the main package emits the disallowed WARNING (aesara.tensor.blas): Using NumPy C-API based implementation for BLAS functions.

I just added that import so that we can check if the warning is triggered on other platforms. (Nope, all good ✔️)

Next I think we can remove couldn't find executable from the allowed warnings. (All good ✔️)

Then we need to make sure we get to the root cause of that warning... or simply discontinue pypy 3.7 support, since pypy 3.8 and 3.9 are now supported on Conda-Forge. (I could start that migration if you'd like, but one thing at a time...)

@maresb
Copy link
Contributor Author

maresb commented Jul 13, 2022

This looks really good to me from the perspective of the warnings. I think the problem is solved.

Regarding Using NumPy C-API based implementation for BLAS functions on pypy 3.7 on Windows, I'm inclined to drop support for pypy 3.7. I don't think it's being actively maintained on Conda-Forge now that 3.8 and 3.9 are out.

@maresb
Copy link
Contributor Author

maresb commented Jul 22, 2022

This has been cherry-picked into #95.

@maresb maresb closed this Jul 22, 2022
@rzyu45
Copy link

rzyu45 commented Jul 27, 2022

Thanks to the contributors for their efforts in addressing the issue. Now it works well on my windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants