Hi again,
A small fix is needed for the
0004-Added-system-test-for-reading-of-a-multi-column-form.patch file
(git hash 034ec73171270182afe6c4907179d46eebe1b759). Spaces are
needed after the commas in the self.assertAlmostEqual() calls. The
2to3 conversion program (for Python 2 to Python 3 conversion)
highlights this issue. Also, the first few tests should be against
floats rather than integers, i.e. simply add '.0' to the end of each
number.
Cheers,
Edward
On 2 August 2013 17:39, Edward d'Auvergne <edward@xxxxxxxxxxxxx> wrote:
Hi Troels,
I'm getting through you patches and have applied the first two. These
are ok. I have an issue with
0003-The-ID-of-spins-in-seriesTab_multi.ser-was-not-forma.patch though
(git hash f49b4f096773cb2f7ecb70293122e03b0b40e889). It says that you
are changing the seriesTab_multi.ser file's 'ASS' column format to
that of Sparky. Why is this so? Is there an NMRPipe seriesTab format
which is different from Sparky? If so, we should support that.
Cheers,
Edward
On 2 August 2013 17:29, Troels E. Linnet
<NO-REPLY.INVALID-ADDRESS@xxxxxxx> wrote:
Follow-up Comment #53, sr #3043 (project relax):
Multi colums import now supported.
Note, I made it possible to autogenerate spectrum ID's, if
spectrum_id='auto' for the function:
spectrum.read_intensities(file="seriesTab_multi.ser",
dir=status.install_path
+ sep+'test_suite'+sep+'shared_data'+sep+'peak_lists', spectrum_id='auto',
int_method='point sum')
(file #18619)
_______________________________________________________
Additional Item Attachment:
File name: stab.patch.tar.gz Size:4 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/support/?3043>
_______________________________________________
Message sent via/by Gna!
http://gna.org/