search

NNPDF / pineappl

PineAPPL is not an extension of APPLgrid
https://nnpdf.github.io/pineappl/
GNU General Public License v3.0
12 stars 3 forks source link

Handle zero EKO passed to `Grid::evolve` #204

Closed felixhekhorn closed 1 year ago

felixhekhorn commented 1 year ago

As @andreab1997 detected: when evolving with an EKO which is all 0s (by mistake) this results in an invalid FK table with no orders (and so the NonTrivialOrder error is returned). The new evolve somewhere discards operators which are 0 everywhere to gain speed - this is just fine.

However, if the EKO is all 0s we should either complain (maybe preferred) or return a valid FK table which is 0 everywhere.

cschwan commented 1 year ago

I agree, and I will take care of that. Note that all fixes will go into version 0.6.x.

cschwan commented 1 year ago

Should be fixed in commit 91924356b1a2b0db4d9ee6036222ecb4a595cc40. Reopen if that isn't the case.