conda-forge / scipy-feedstock

A conda-smithy repository for scipy.
BSD 3-Clause "New" or "Revised" License
6 stars 36 forks source link

SciPy 1.12.0 #262

Closed h-vetinari closed 9 months ago

h-vetinari commented 10 months ago

~Let's try to build the release candidate~

Closes #263

conda-forge-webservices[bot] commented 10 months 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.

h-vetinari commented 10 months ago

It looks like boost is using an unfortunate choice of template variable name on windows, which conflicts with a macro:

In file included from ..\scipy\_lib\boost_math\include\boost/math/policies/policy.hpp:11:
..\scipy\_lib\boost_math\include\boost/math/tools/mp.hpp(171,34): error: expected parameter declarator
  171 | template<typename L, std::size_t I>
      |                                  ^
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\ucrt\complex.h(63,20): note: expanded from macro 'I'
   63 | #define I          _Complex_I
      |                    ^
C:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\ucrt\complex.h(62,29): note: expanded from macro '_Complex_I'
   62 | #define _Complex_I _FCbuild(0.0F, 1.0F)
      |                             ^
andyfaff commented 10 months ago

Seems that boost should probably change their macro if it shadows something further upstream?

github-actions[bot] commented 9 months ago

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

I considered the following status checks when analyzing this PR:

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

maresb commented 9 months ago

Only on ppc64le:

FAILED stats/tests/test_distributions.py::TestLaplace::test_fit_MLE_comp_optimizer[0.5-0.2] - assert (51.80214127750128 < 51.802141277501164 or False)
 +  where False = <function allclose at 0x10001f113df0>(51.80214127750128, 51.802141277501164, atol=1e-15, rtol=1e-15)
 +    where <function allclose at 0x10001f113df0> = np.allclose
maresb commented 9 months ago

Looks like the RNG for Laplace is missing the seed. I might be missing something but I bet it will succeed on rerun.

https://github.com/scipy/scipy/blob/44b39e1e740795aa8cd4d69e88d6c0acb116eeac/scipy/stats/tests/test_distributions.py#L2999

maresb commented 9 months ago

After checking the results of the build-upload job, I noticed that there was a failure in ppc64le / Python 3.12 with the same numbers as the failure above on ppc64le / Python 3.11. I wrote a note about it here. Hopefully my upstream PR resolves this for scipy >=1.12.1.