[kepler-dev] ParameterSet

[Edward A. Lee]

I am (finally) checking in a fix to the bug described below.
The bug was in the Variable class.  The problem was that if you
changed the name of a Variable (which is what you do when you
drop a Parameter on screen and then change its name), the variable
with a new name may now shadow some other variable (such as one
in a ParameterSet or ScopeExtendingAttribute).  But those now
shadowed variables were not getting invalidated, and hence the
presence of the new variable would not be noticed until you saved
and re-loaded the model.

Thanks Norbert for pointing out this (rather subtle) bug.

Edward

At 10:33 AM 10/9/2006, Christopher Brooks wrote:
 >Hi Norbert,
 >Sorry about the bounce, I added you to my no bounce list.
 >
 >I think Edward might have some say on ParameterSet.
 >
 >_Christopher
 >
 >--------
 >     > From: Norbert Podhorszki <pnorbert at cs.ucdavis.edu>
 >     > To: Christopher Brooks <cxh at eecs.berkeley.edu>
 >     > Subject: Re: ParameterSet
 >
 >     >
 >     > Hi Christopher,
 >     > What is the current status of ParameterSet?
 >     > Its doc does not tell that the parameters read from file 
must not be on
 >     > screen. So what is the case?
 >     >
 >     > Current ptolemy behaves strangely if I have a parameter both
 > on screen an
 >    d
 >     > in file:
 >     > - First edit and run: whichever is created first, will be in effect.
 >     > - Open existing file: the parameter in file has no effect.
 > The on-screen
 >     > parameter is used and its value does not become updated.
 >     >
 >     > Shall I remove the on-screen parameters once I want to use
 > config files?
 >     >
 >     > I liked much better the unambigous practice of olden times 
(last June):
 >     > The config file had priority over on-screen values all the time.
 >     >
 >     > Norbert
 ><<attachments removed>>
 >_______________________________________________
 >Kepler-dev mailing list
 >Kepler-dev at ecoinformatics.org
 >http://mercury.nceas.ucsb.edu/ecoinformatics/mailman/listinfo/kepler-dev

------------
Edward A. Lee
Chair of EECS and Robert S. Pepper Distinguished Professor
231 Cory Hall, UC Berkeley, Berkeley, CA 94720-1770
phone: 510-642-0253, fax: 510-642-2845
eal at eecs.Berkeley.EDU, http://ptolemy.eecs.berkeley.edu/~eal

_______________________________________________
Kepler-dev mailing list
Kepler-dev at ecoinformatics.org
http://mercury.nceas.ucsb.edu/ecoinformatics/mailman/listinfo/kepler-dev

------------
Edward A. Lee
Chair of EECS and Robert S. Pepper Distinguished Professor
231 Cory Hall, UC Berkeley, Berkeley, CA 94720-1770
phone: 510-642-0253, fax: 510-642-2845
eal at eecs.Berkeley.EDU, http://ptolemy.eecs.berkeley.edu/~eal