conda-forge / petsc4py-feedstock

A conda-smithy repository for petsc4py.
BSD 3-Clause "New" or "Revised" License
0 stars 15 forks source link

consider thinning supported matrix #96

Closed minrk closed 2 days ago

minrk commented 3 months ago

petsc4py and slepc4py now have 260 build variants because of:

This is causing rerenders to take at least 30 minutes, and time out on the bot, so I don't think any automated PRs are going to be made to these repos until something changes.

I think we should consider culling some of these to keep things manageable. Options include:

It would really be ideal if petsc4py didn't need to be aware of the cuda build variants at all, but I think @stefanozampini confirmed that it doesn't work that way (I can't find the reference anymore, though).

stefanozampini commented 3 months ago

Unfortunately it's not possible at the moment to use a petsc at runtime which is different than the build one

dalcinl commented 3 months ago

@stefanozampini Could you please remember me why we have 3 CUDA variants?

stefanozampini commented 3 months ago

The cuda variants are 11.8, 12.4 and nocuda

minrk commented 3 months ago

Looks like the rerender times are fixable upstream, and thinning the build matrix doesn't actually help rerender time. More info here: https://github.com/regro/cf-scripts/issues/2561 and a couple linked conda-build PRs.

So we should only bother thinning here if build time is problematic.

minrk commented 2 days ago

closing this since rerender time is a lot more manageable now