Opened 11 years ago
Closed 11 years ago
#567 closed defect (fixed)
running alpha_s setup not properly working
Reported by: | Juergen Reuter | Owned by: | kilian |
---|---|---|---|
Priority: | P0 | Milestone: | v2.2.0 |
Component: | core | Version: | 2.1.1 |
Severity: | normal | Keywords: | |
Cc: |
Description
This is probably because of MD5 sums. The file alphas.sin from the test suite gives identical results for the second and third integration (which shouldn't be the case). Changing the order of the alpha_s evolution changes the result of the alpha_s_from_mz case, and accordingly the one from pdf_builtin. So this is broken.
Change History (11)
comment:1 Changed 11 years ago by
Owner: | changed from kilian to Juergen Reuter |
---|---|
Status: | new → assigned |
comment:2 Changed 11 years ago by
Introduced MD5 sums for the QCD setup in r4897. The update of the phase-space works from fixed to non-fixed, but not from non-fixed to running from pdf_builtin. Still have to investigate this. Also, with 4.6.3/4 there is a double free corruption in the alpha_s case, which we have to look into.
comment:3 Changed 11 years ago by
when using alphas.sin from the self tests, in the second case it says "qcd setup has changed" but not for the third integration, why?
comment:4 Changed 11 years ago by
It actually does change, but not if it is three processes, only for the second one :(
comment:5 Changed 11 years ago by
Owner: | changed from Juergen Reuter to kilian |
---|---|
Status: | assigned → new |
comment:7 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
This was actually an error in the test script: the flag ?alpha_s_from_mz
takes precedence over ?alpha_s_from_pdf_builtin
.
comment:8 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Please recheck the phase space MD5 sum setup.
comment:9 Changed 11 years ago by
Priority: | P1 → P0 |
---|
As we talked about that offline... please set the defaults as agreed upon and close the ticket (if really phase space update happens in all cases in the test) (did you run it without the --r option outside the testsuite setup)
comment:10 Changed 11 years ago by
Gotcha ...
It turns out that the QCD setup is always checked for the phase space, but not for the grids. It should be the other way round -- there is no reason why the phase space should depend on the way of computing alpha_s. Will correct this.
comment:11 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Finally -
All over the place, the QCD setup was incorporated in the phase-space configuration, which it should not affect. Instead, it was missing from the process (i.e., integration object) configuration, which should depend on it. This is now corrected: r5017.
Also, we require now that exactly one flag for the QCD alpha calculation mode is set at a time.
We need a QCD MD5 sum, going to do this. 3RD TRY TO TYPE THIS INTO THE FUCKING TRACKER!!!!