Hi Michael, I thought you would have started with your code but if you think it will be easier for you to start with my code rather than your original NOE code, then that will be fine! More comments below: On 24 February 2010 05:48, <michael.bieri@xxxxxx> wrote:
Author: michaelbieri Date: Wed Feb 24 05:48:49 2010 New Revision: 10891 URL: http://svn.gna.org/viewcvs/relax?rev=10891&view=rev Log: All unused coding from Rx-calculations were deleted in auto_noe.py
This commit message is a little confusing and I'm guessing you mean that you converted all the Rx curve-fitting specific parts to be NOE calculation specific. Mentioning that you removed the peak height selection GUI element would also be useful.
Modified: branches/bieri_gui/gui_bieri/analyses/auto_noe.py Modified: branches/bieri_gui/gui_bieri/analyses/auto_noe.py URL: http://svn.gna.org/viewcvs/relax/branches/bieri_gui/gui_bieri/analyses/auto_noe.py?rev=10891&r1=10890&r2=10891&view=diff ============================================================================== --- branches/bieri_gui/gui_bieri/analyses/auto_noe.py (original) +++ branches/bieri_gui/gui_bieri/analyses/auto_noe.py Wed Feb 24 05:48:49 2010 @@ -32,23 +32,22 @@ import wx # relax module imports. -from auto_analyses.relax_fit import Relax_fit +from auto_analyses.noe import NOE_calc from data import Relax_data_store; ds = Relax_data_store() # relaxGUI module imports. from gui_bieri.analyses.project import open_file from gui_bieri.base_classes import Container -from gui_bieri.components.spectrum import Peak_intensity from gui_bieri.controller import Redirect_text, Thread_container from gui_bieri.derived_wx_classes import StructureTextCtrl -from gui_bieri.filedialog import multi_openfile, opendir +from gui_bieri.filedialog import multi_openfile, opendir, openfile from gui_bieri.message import error_message -from gui_bieri.paths import ADD_ICON, CANCEL_ICON, IMAGE_PATH, REMOVE_ICON - - - -class Auto_rx: - """The base class for the R1 and R2 frames.""" +from gui_bieri.paths import IMAGE_PATH + + + +class Auto_noe: + """The base class for the noe frames.""" # Hardcoded variables. analysis_type = None @@ -56,7 +55,7 @@ label = None def __init__(self, gui, notebook, hardcoded_index=None): - """Build the automatic R1 and R2 analysis GUI frame elements. + """Build the automatic NOE analysis GUI frame elements. @param gui: The main GUI class. @type gui: gui_bieri.relax_gui.Main instance @@ -321,9 +320,6 @@ # Add the unresolved spins GUI element. self.add_unresolved_spins(box) - # Add the peak list selection GUI element. - self.peak_intensity = Peak_intensity(gui=self.gui, parent=self.parent, data=self.data, label=self.label, box=box) -
This is the main reason why shifting GUI elements into their own classes in gui_bieri/components is very useful. Adding and removing them to the analysis specific frames is trivial! Cheers, Edward