Opened 11 years ago
Closed 10 years ago
#628 closed defect (fixed)
LEF reading: Scale and alpha-QCD input
Reported by: | kilian | Owned by: | kilian |
---|---|---|---|
Priority: | P3 | Milestone: | v2.2.5 |
Component: | core | Version: | 2.2.0beta |
Severity: | normal | Keywords: | |
Cc: |
Description
Apparently not yet functional. To be checked.
Change History (15)
comment:1 Changed 11 years ago by
comment:2 follow-up: 3 Changed 11 years ago by
?update_alpha_s
and ?update_scale
have benn used in 2.1 to change the alpha_s and scales for rescanning events. In 2.2 this seems to be done always if according expressions are specified. This seems more uer-friendly. So if really not needed these variables should be eliminated.
comment:3 Changed 11 years ago by
Replying to jr_reuter:
?update_alpha_s
and?update_scale
have benn used in 2.1 to change the alpha_s and scales for rescanning events. In 2.2 this seems to be done always if according expressions are specified. This seems more uer-friendly. So if really not needed these variables should be eliminated.
Good, so remove those. I was not sure.
comment:4 Changed 11 years ago by
Hm, I'm also not totally sure. Could someone double/triple-check? If confirmed, also ?update_parameters is irrelevant. What about the reading from LHE files?
comment:5 Changed 11 years ago by
Yes, it does: update_sqme triggers evaluate_trace which contains via term_instance_evaluate_interaction the subroutine process_compute_amplitude: this resets alpha_s. So check. Directly before this routine call in process_instance_recover there is the call of term%instance%evaluate_expressions which expands and sets the scale expressions. SO they are modified if changed. This is called as event_recalculate (inherited to entry%recalculate in simulation%recalculate which is called both in cases of rescanning and of an alt_entry.
comment:7 Changed 11 years ago by
Milestone: | v2.2.0 → v2.2.1 |
---|
comment:8 Changed 11 years ago by
Actually, in the subroutine hepeup_to_event there is this code (commented out):
!!! Not implemented yet: ! if (scale > 0) call event%set_scales (scale) ! if (alpha_qcd > 0) call event%set_alpha_qcd (alpha_qcd)
Related to this?
comment:9 Changed 11 years ago by
Milestone: | v2.2.2 → v2.2.3 |
---|
comment:10 Changed 11 years ago by
Milestone: | v2.2.3 → v2.2.4 |
---|
comment:11 Changed 10 years ago by
Status: | new → assigned |
---|
While #701 is pending, I'll try to resolve this ticket. Plan: Parameters ?use_alphas_from_file
and ?use_scale_from_file
which tell WHIZARD to read the values in the file and use them in the calculation instead of the current environment setting. By default, the entries in the file are ignored. All of this applies only if ?update_sqme
is true, and only if the file format supports those entries.
comment:12 Changed 10 years ago by
Enabled ?use_alpha_qcd_from_file
for LHEF, in r6574.
To be done: enable for HEPMC and LCIO, and analogously for ?use_scale_from_file.
comment:13 Changed 10 years ago by
With r6578, the feature is implemented for LHEF, HepMC, and LCIO.
I'd close the ticket, except for
- how to name the new user-level parameters? I never know whether to write alpha_qcd or alpha_s (should probably stick to the latter).
- I believe that the LCIO code is correct (exactly analogous to HepMC) but couldn't test it, also didn't write unit tests - my runtime linker doesn't want to link the LCIO shared lib.
comment:14 follow-up: 15 Changed 10 years ago by
Before you close the ticket, please do not forget to add the options to the manual, the main text and the SINDARIN reference list!
comment:15 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Replying to jr_reuter:
Before you close the ticket, please do not forget to add the options to the manual, the main text and the SINDARIN reference list!
Done in r6579. Also, reverted the parameter name to ?use_alpha_s_from_file
, consistent with other parameters. I skip the missing LCIO test cases and close this ticket.
There's also the
?update_scale
and?update_alpha_s
parameters which are currently disabled.