New Multi-Closure test analysis module
addresses #2159
The main goals of this PR are:
- multiclosure analysis modules are reviewed and rewritten based on the new findings of the inconsistent closure tests paper.
- All functions / helpers used for computation of bias with exp. covmat are removed.
- All functions / helpers for computation of variance are removed.
- Removed vp-comparefits --closure
@jacoterh will you review this one?
Yes, sorry about the delay - I will take a closer look
Hi @comane, many thanks for this. Would you mind updating the corresponding documentation on the website as well? Specifically, could you make sure what is written on https://github.com/NNPDF/nnpdf/blob/3556b8f13f8284a6ba06fd82f1b0541080672370/doc/sphinx/source/tutorials/closureestimators.rst is still up to date? For example, the --closure option was removed, while the documentation mentions it.
What is the status of this one?
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 go.
What is the status of this one?
What is the status of this one?