So what Tim has done is to protect Analysis.py against this new C function
not existing. But if it does not exist then the argServer might have
currentPeaks() with deleted peaks in it, so that is dangerous. So to be
100% safe (if you want to use macros which use currentPeaks()) it is best
to update that C code I sent in the email a day or two ago (and
re-compile of course by typing "make"). We could put the C code on the
update server as well, only people might not know to download that if they
download Analysis.py. So this points out the real issue with all this
update stuff: there could be dependencies. I'm not sure how we're going
to deal with that issue, I'm not sure we want to re-invent CVS.
Wayne
On Mon, 24 Oct 2005, Tim Stevens wrote:
> > "/home/justin/Programs/ccpnmr/ccpnmr1.0/python/ccpnmr/analysis/Analysis.py",
> > line 540, in removeCPeak
> > if peak.cPeak.getIsSelected():
> > AttributeError: getIsSelected
>
> Update posted to eliminate this problem, but top.currentPeaks() may still
> be out of date.
>
> T.
>
> -------------------------------------------------------------------------------
> Dr Tim Stevens Email: [log in to unmask]
> Department of Biochemistry [log in to unmask]
> University of Cambridge Phone: +44 1223 766022 (office)
> 80 Tennis Court Road +44 7816 338275 (mobile)
> Old Addenbrooke's Site +44 1223 364613 (home)
> Cambridge CB2 1GA WWWeb: http://www.bio.cam.ac.uk/~tjs23
> United Kingdom http://www.pantonia.co.uk
> -------------------------------------------------------------------------------
> ------ +NH3CH(CH(CH3)OH)C(O)NHCH(CH(CH3)CH2CH3)C(O)NHCH(CH2CH2SCH3)CO2- -------
> -------------------------------------------------------------------------------
>
|