8000 Make the diagonal fit the default · Issue #2332 · NNPDF/nnpdf · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Make the diagonal fit the default #2332

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
scarlehoff opened this issue Jun 16, 2025 · 0 comments
8090
Open

Make the diagonal fit the default #2332

scarlehoff opened this issue Jun 16, 2025 · 0 comments
Assignees
@jacoterh

Comments

@scarlehoff
Copy link
Member

It has been decided that for 4.1 we will want the diagonal covmat to be the default way in which fits are run, see this comment:
#2301 (comment)

(it would be best if this is done as one of the first PR after 4.1.0 is released, so by looking at the version of the code one can redo previous fits with or without diagonal)

In addition to make it default, we want to make sure that by using the diagonal basis we reduce the cost on memory of the fits, so that PR would be the perfect place to benchmark that and improve the performance of the fits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacoterh jacoterh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@scarlehoff @jacoterh
0