Re: [bug #22071] relax unit test and system test not functional -- May 23, 2014 - 11:36

Hi Ed.

Its solved!

command: time ./relax -u

Linux svn: 15.346u 0.412s 0:16.56 95.1%
Linux git: 15.376u 0.362s 0:16.38 96.0%

Mac svn: real 0m27.724s user 0m27.020s sys 0m0.585s
Mac git: real 0m29.684s user 0m28.672s sys 0m0.942s

2014-05-23 11:28 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:
> Hi,
>
> Would it be possible to create a quick text table of the timings for
> './relax -s Relax_disp.test_cpmg_synthetic_dx_map_points', './relax -s
> Relax_disp', './relax -s', './relax -u' for both git-svn and svn, for
> both the current revision and then going back to revision r23289 (to
> match relax 3.2.0).  I.e. a 4x4 table.  This will help to work out
> what has changed.  Note that I have just committed a speed up to the
> version module (r23373,
> http://article.gmane.org/gmane.science.nmr.relax.scm/21119).  Things
> will be a little slower now on your git-svn code, as relax is pulling
> out the repository information whereas before these version module
> functions were simply bypassed.
>
> Cheers!
>
> Edward
>
>
>
> On 23 May 2014 11:15, Troels Emtekær Linnet <tlinnet@xxxxxxxxxxxxx> wrote:
> > Hi Ed.
> >
> > I am now running svn rep.
> > I can run single test:
> > relax_trunk_svn$ ./relax -s Relax_disp.test_cpmg_synthetic_dx_map_points
> >
> > Here there about 4 seconds delay, before first dot.
> > relax_trunk_svn$ ./relax -s Relax_disp
> >
> > sh, svn, python rapidly changing in top of terminal.
> >
> > Here there about 9-10 seconds delay, before first dot.
> > relax_trunk_svn$ ./relax -s
> >
> > Here there about 11-12 seconds delay, before first dot.
> > relax_trunk_svn$ ./relax -u
> >
> > in git:
> > 3 seconds delay, perl 5.16, sh, pythonn blinking
> > relax_trunk$ ./relax -s Relax_disp.test_cpmg_synthetic_dx_map_points
> >
> > about 24 seconds delay, perl 5.16, sh, pythonn blinking
> > relax_trunk$ ./relax -s Relax_disp
> >
> > I will try go back
> >
> >
> > 2014-05-23 11:01 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:
> > > You might have to help me work this one out, as everything works for
> > > me on all test systems.  What exactly do you see?  Can you run
> > > individual tests:
> > >
> > > $ ./relax -s Relax_disp.test_cpmg_synthetic_dx_map_points
> > >
> > > How long do you wait?  Does the test suite hang indefinitely, or is it
> > > just slower to start?  Is this only on your git-svn code, or svn as
> > > well?  Can you go back to r23342, i.e. before the git-svn info
> > > changes, and see if the problem is there.  Or back to r23359, just
> > > before the test suite printout changes, and check at that point too.
> > >
> > > Cheers!
> > >
> > > Edward
> > >
> > >
> > > On 23 May 2014 10:46, Troels Emtekær Linnet <tlinnet@xxxxxxxxxxxxx> wrote:
> > > > Yup:
> > > >
> > > > relax_disp_speed -s
> > > > tlinnet@linmac:disp_speed$ cd $s; git svn info
> > > > Path: .
> > > > URL: svn+ssh://svn.gna.org/svn/relax/branches/disp_speed
> > > > Repository Root: svn+ssh://svn.gna.org/svn/relax
> > > > Repository UUID: b7916896-f9f9-0310-9fe5-b3996d8957d5
> > > > Revision: 23372
> > > > Node Kind: directory
> > > > Schedule: normal
> > > > Last Changed Author: tlinnet
> > > > Last Changed Rev: 23372
> > > > Last Changed Date: 2014-05-23 10:30:49 +0200 (Fri, 23 May 2014)
> > > >
> > > > 2014-05-23 10:42 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:
> > > > > Hi,
> > > > >
> > > > > Cheers.  The change I made at r23366
> > > > > (http://article.gmane.org/gmane.science.nmr.relax.scm/21112) however
> > > > > replaces this with:
> > > > >
> > > > > $ cd $s; git svn info
> > > > >
> > > > > Does this work for you?
> > > > >
> > > > > Regards,
> > > > >
> > > > > Edward
> > > > >
> > > > >
> > > > >
> > > > > On 23 May 2014 10:38, Troels Emtekær Linnet <tlinnet@xxxxxxxxxxxxx> 
> > > > > wrote:
> > > > > > Hi Ed.
> > > > > >
> > > > > > It still does not work:
> > > > > > But
> > > > > >
> > > > > > relax> from status import Status; status = Status()
> > > > > > relax> print status.install_path
> > > > > > /Users/tlinnet/software/disp_speed
> > > > > > relax>
> > > > > > tlinnet@linmac:disp_speed$ s=/Users/tlinnet/software/disp_speed
> > > > > > tlinnet@linmac:disp_speed$ cd $s; git svn find-rev $(git rev-parse 
> > > > > > HEAD)
> > > > > > 23372
> > > > > >
> > > > > > 2014-05-23 9:21 GMT+02:00 Edward d'Auvergne <edward@xxxxxxxxxxxxx>:
> > > > > > > Hi,
> > > > > > >
> > > > > > > I just made another simplification for the git-svn version.py support:
> > > > > > >
> > > > > > > r23366:  http://article.gmane.org/gmane.science.nmr.relax.scm/21112
> > > > > > >
> > > > > > > Once this issue is resolved, I'll look at releasing relax 3.2.1 with
> > > > > > > all the recent bugfixes.
> > > > > > >
> > > > > > > Cheers,
> > > > > > >
> > > > > > > Edward
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On 23 May 2014 09:12, Edward d'Auvergne <edward@xxxxxxxxxxxxx> wrote:
> > > > > > > > Hi Troels,
> > > > > > > >
> > > > > > > > If that change doesn't work, could you try running:
> > > > > > > >
> > > > > > > > $ cd %s; git svn find-rev $(git rev-parse HEAD)
> > > > > > > >
> > > > > > > > where the %s is replaced by the value of status.install_path.  This 
> > > > > > > > is
> > > > > > > > the command in the version file for obtaining the subversion revision
> > > > > > > > number from a git-svn copy.  Or:
> > > > > > > >
> > > > > > > > $ cd %s; git svn info
> > > > > > > >
> > > > > > > > This is the command used to find the repository URL.  These two bits
> > > > > > > > of information are used for the relax intro text, which used to look
> > > > > > > > like:
> > > > > > > >
> > > > > > > > """
> > > > > > > >
> > > > > > > >                                   relax repository checkout rNone
> > > > > > > >                                                 None
> > > > > > > >
> > > > > > > >                               Molecular dynamics by NMR data analysis
> > > > > > > >
> > > > > > > >                              Copyright (C) 2001-2006 Edward 
> > > > > > > > d'Auvergne
> > > > > > > >                          Copyright (C) 2006-2014 the relax 
> > > > > > > > development team
> > > > > > > >
> > > > > > > > """
> > > > > > > >
> > > > > > > > But this new git-svn support:
> > > > > > > >
> > > > > > > > """
> > > > > > > >
> > > > > > > >                                   relax repository checkout r23365
> > > > > > > >                                svn+ssh://svn.gna.org/svn/relax/trunk
> > > > > > > >
> > > > > > > >                               Molecular dynamics by NMR data analysis
> > > > > > > >
> > > > > > > >                              Copyright (C) 2001-2006 Edward 
> > > > > > > > d'Auvergne
> > > > > > > >                          Copyright (C) 2006-2014 the relax 
> > > > > > > > development team
> > > > > > > >
> > > > > > > > """
> > > > > > > >
> > > > > > > > Maybe there is a shell incompatibility with one of these commands, so
> > > > > > > > make sure you use the 'cd %s; ' part to check that too.
> > > > > > > >
> > > > > > > > Cheers,
> > > > > > > >
> > > > > > > > Edward
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > On 23 May 2014 09:05, Edward d'Auvergne <edward@xxxxxxxxxxxxx> wrote:
> > > > > > > > > Hi,
> > > > > > > > >
> > > > > > > > > I first thought that it might have been this test suite printout 
> > > > > > > > > change:
> > > > > > > > >
> > > > > > > > > r23360:  http://article.gmane.org/gmane.science.nmr.relax.scm/21105
> > > > > > > > >
> > > > > > > > > But now, with that last message, it's clear that it is these two to
> > > > > > > > > the version.py module:
> > > > > > > > >
> > > > > > > > > r23345:  http://article.gmane.org/gmane.science.nmr.relax.scm/21088
> > > > > > > > > r23343:  http://article.gmane.org/gmane.science.nmr.relax.scm/21090
> > > > > > > > >
> > > > > > > > > The git-svn support for extracting the repository information I 
> > > > > > > > > added
> > > > > > > > > mustn't be functioning as I expected.  I've attempted a fix at:
> > > > > > > > >
> > > > > > > > > r23365:  http://article.gmane.org/gmane.science.nmr.relax.scm/21111
> > > > > > > > >
> > > > > > > > > Cheers,
> > > > > > > > >
> > > > > > > > > Edward
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > On 22 May 2014 22:58, Troels E. Linnet 
> > > > > > > > > <NO-REPLY.INVALID-ADDRESS@xxxxxxx> wrote:
> > > > > > > > > > Follow-up Comment #2, bug #22071 (project relax):
> > > > > > > > > >
> > > > > > > > > > If I break, this comes:
> > > > > > > > > >
> > > > > > > > > >   File "./relax", line 7, in <module>
> > > > > > > > > >     relax.start(profile_flag=False)
> > > > > > > > > >   File 
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/relax.py", line 
> > > > > > > > > > 118,
> > > > > > > > > > in start
> > > > > > > > > >     processor.run()
> > > > > > > > > >   File 
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/multi/processor.py",
> > > > > > > > > > line 494, in run
> > > > > > > > > >     self.callback.init_master(self)
> > > > > > > > > >   File 
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/multi/__init__.py",
> > > > > > > > > > line 318, in default_init_master
> > > > > > > > > >     self.master.run()
> > > > > > > > > >   File 
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/relax.py", line 
> > > > > > > > > > 246,
> > > > > > > > > > in run
> > > > > > > > > >     runner.run_unit_tests()
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/test_suite_runner.py",
> > > > > > > > > > line 192, in run_unit_tests
> > > > > > > > > >     self.unit_result = unit_runner.run(runner=self.runner)
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/unit_tests/unit_test_runner.py",
> > > > > > > > > > line 624, in run
> > > > > > > > > >     finder.scan_paths()
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/unit_tests/unit_test_runner.py",
> > > > > > > > > > line 378, in scan_paths
> > > > > > > > > >     test_case = load_test_case(dir_path, module_name, class_name)
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/unit_tests/unit_test_runner.py",
> > > > > > > > > > line 316, in load_test_case
> > > > > > > > > >     return TestLoader().loadTestsFromTestCase(clazz)
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/relax_test_loader.py",
> > > > > > > > > > line 56, in loadTestsFromTestCase
> > > > > > > > > >     test_case = testCaseClass(testCaseNames[i])
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/test_suite/unit_tests/_prompt/test_spectrum.py",
> > > > > > > > > > line 43, in __init__
> > > > > > > > > >     self.interpreter = Interpreter(show_script=False, 
> > > > > > > > > > raise_relax_error=True)
> > > > > > > > > >   File
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/prompt/interpreter.py",
> > > > > > > > > >  line
> > > > > > > > > > 87, in __init__
> > > > > > > > > >     self.__intro_string = info.intro_text()
> > > > > > > > > >   File "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/info.py", 
> > > > > > > > > > line 287,
> > > > > > > > > > in intro_text
> > > > > > > > > >     text2 = "%s" % (url())
> > > > > > > > > >   File 
> > > > > > > > > > "/sbinlab2/tlinnet/software/NMR-relax/relax_trunk/version.py", line
> > > > > > > > > > 120, in url
> > > > > > > > > >     if not pipe.stderr.readlines():
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >     _______________________________________________________
> > > > > > > > > >
> > > > > > > > > > Reply to this item at:
> > > > > > > > > >
> > > > > > > > > >   <http://gna.org/bugs/?22071>
> > > > > > > > > >
> > > > > > > > > > _______________________________________________
> > > > > > > > > >   Message sent via/by Gna!
> > > > > > > > > >   http://gna.org/
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > _______________________________________________
> > > > > > > > > > relax (http://www.nmr-relax.com)
> > > > > > > > > >
> > > > > > > > > > This is the relax-devel mailing list
> > > > > > > > > > relax-devel@xxxxxxx
> > > > > > > > > >
> > > > > > > > > > To unsubscribe from this list, get a password
> > > > > > > > > > reminder, or change your subscription options,
> > > > > > > > > > visit the list information page at
> > > > > > > > > > https://mail.gna.org/listinfo/relax-devel