Mark Nestor Costantini

The C-factor is CF_QCD_ATLAS_DY_7TEV_CC in theory 400, and in particular from row 7 to 19 ********************************************************************************SetName: ATLASWZRAP11CC Author: Emanuele R. Nocera [email protected] Date: August 2021 CodesUsed: MCFM v8 TheoryInput: NNPDF4.0...

Hi @cschwan, sorry for the confusion I realise my comment might have been unclear. I am looking at row 7 to 19 of the CFactor I have printed. Those values...

> are all the functions affected by this change already unit tested? No to my understanding the closuretests module is not really covered by tests right now.

Once rebased, can this be merged? Would also close the PR below #79

> Dear [@comane](https://github.com/comane) can you please remind me where the tables named `atlas_inclusive_*` in here https://github.com/NNPDF/nnpdf/tree/master/nnpdf_data/nnpdf_data/commondata/ATLAS_1JET_8TEV_R06/rawdata come from? Thanks. Dear @enocera, I believe they can be found in the extra...

> Also, why is the statistical uncertainty `MULT` here below? https://github.com/NNPDF/nnpdf/tree/master/nnpdf_data/nnpdf_data/commondata/ATLAS_1JET_8TEV_R06/rawdata Shouldn't it be `ADD`? I am not sure, I think that might be a mistake

> Note that postfit writes a `FromMCReplica`, which allows you to know which fitted replica a given PDF replica corresponds to. Of course if you don't like the sorting it...

Yes I agree, the PR #1982 that is used for the analysis is almost ready to be merged. I will take care of it before starting anything else.

> What is the status of this one? I wanted #2258 to be merged first. Now will have a final look at it. However, it should be basically ready to...

Hi @RoyStegeman, can you perhaps give more details on why this is a bug. Should a different formula be used for the computation of how much the sum rules for...