whizard is hosted by Hepforge, IPPP Durham

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 Juergen Reuter

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...

comment:2 Changed 12 years ago by kilian

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 Juergen Reuter

Ok, I discovered the very same thing a couple of days ago and dumped this gain of insight into ticket #25. WK, should we close one of these two tickets, then? Or does #25 contain more info than this one here!?

comment:4 Changed 11 years ago by Juergen Reuter

Milestone: v2.2.1v2.4.0

comment:5 Changed 11 years ago by Juergen Reuter

Milestone: v2.4.0v2.2.4

comment:6 Changed 10 years ago by Juergen Reuter

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 kilian

Resolution: fixed
Status: newclosed

Fixed, test added in r6314.

Note: See TracTickets for help on using tickets.