whizard is hosted by Hepforge, IPPP Durham

Opened 10 years ago

Closed 10 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 10 years ago by Juergen Reuter

Owner: changed from kilian to Juergen Reuter
Status: newassigned

We need a QCD MD5 sum, going to do this. 3RD TRY TO TYPE THIS INTO THE FUCKING TRACKER!!!!

comment:2 Changed 10 years ago by Juergen Reuter

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 10 years ago by Juergen Reuter

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 10 years ago by Juergen Reuter

It actually does change, but not if it is three processes, only for the second one :(

comment:5 Changed 10 years ago by Juergen Reuter

Owner: changed from Juergen Reuter to kilian
Status: assignednew

comment:6 Changed 10 years ago by Juergen Reuter

Maybe connected to #538.

comment:7 Changed 10 years ago by kilian

Resolution: fixed
Status: newclosed

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 10 years ago by Juergen Reuter

Resolution: fixed
Status: closedreopened

Please recheck the phase space MD5 sum setup.

comment:9 Changed 10 years ago by Juergen Reuter

Priority: P1P0

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 10 years ago by kilian

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 10 years ago by kilian

Resolution: fixed
Status: reopenedclosed

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.

Note: See TracTickets for help on using tickets.