Open adriendelsalle opened 2 years ago
Did you try with gcc>=11.2?
I think it's a minimum requirement:
Yes I understand the perspective to get better performance using std::reduce
. Maybe it would be a good thing to start a developer documentation to mention minimum version of gcc
.
The binding of MeridionalGrid<T>::operator()
still needs to be wrapped in a lambda. We could investigate later how to remove that limitation (returning a template parameter already works).
Yes didn't find a workaround either for MeridionalGrid
Description
I got some compilation errors when compiling the Python bindings on
linux-64
:the
auto
deduction of return type in multiple overloads causespybind11::overload_cast
compilation error onMeridionalGrid<T>::operator()
.gcc=11
should help on that (see the comment I left on relatedpybind11
issue) but it looks like it's not for our case. I would need to spend more time on that to understand what's going onthe reducer used in
gbs-mesh/gbs::msh_curves_set_sizes
causes a compilation error usinggcc=10
gcc>=11.2
doesn't complain. I found a bug instd::reduce
overload using a binary functor ingcc=10
, tracked hereconda
toolchain usinggcc=10
it would be better to usestd::accumulate
instead ofstd::reduce
in upstream projectgbs
, which is not buggy