Opened 12 years ago
Closed 10 years ago
#508 closed defect (fixed)
Name clashes of model parameters
Reported by: | kilian | Owned by: | kilian |
---|---|---|---|
Priority: | P4 | Milestone: | v2.2.6 |
Component: | core | Version: | 2.1.1 |
Severity: | normal | Keywords: | |
Cc: |
Description
When the model 'SM' is preloaded, switching to 'QED' has the effect that the independent parameter 'ee' cannot be set in the Sindarin file. This is because 'ee' in the SM is a derived parameter.
This problem has been present in previous WHIZARD versions as well, it just surfaced in the qedtest_X functional tests of the new version.
The command-line option --no-model now suppresses model preloading, so the name clash can be avoided. However, there should be a generic solution.
Change History (7)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
No, here the problem is that a free parameter cannot replace a derived parameter. The 'locked' attribute is sticky where it shouldn't be. This is caused by the current way model variables are implemented.
Admittedly, this is a rare case and a solution might be postponed to a more general Sindarin overhaul. But I note it here for completeness.
comment:3 Changed 11 years ago by
comment:4 Changed 11 years ago by
Milestone: | v2.2.1 → v2.4.0 |
---|
comment:5 Changed 11 years ago by
Milestone: | v2.4.0 → v2.2.4 |
---|
comment:6 Changed 10 years ago by
Actuallu, with the refactoring of the model parameters, is this problem mentioned in this ticket still existing, or is this solved now?
comment:7 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed, test added in r6314.
We discussed this already, didn't we? Solutions are to either precede the model name (which is awkward for the user, or to use namespaces somehow...