UGENE Forum
https://forum.ugene.net/forum/YaBB.pl
General Category >> Bugs and Issues >> Suggestion for minor change in how UGENE reads Clustal alignments
https://forum.ugene.net/forum/YaBB.pl?num=1238694990
Message started by David Spencer on Apr 3rd, 2009 at 12:56am

Title: Suggestion for minor change in how UGENE reads Clustal alignments
Post by David Spencer on Apr 3rd, 2009 at 12:56am
Hi,

We just discovered (and I verified by going through the source code) that when UGENE imports a clustal 'aln' file it checks the first line for the string "CLUSTAL W". Unfortunately 'clustalx' (the gui front end to 'clustalw') uses the identifier 'CLUSTAL X' here so UGENE imports it but the data gets messed up when it is displayed.
I assume that UGENE could look for simply 'CLUSTAL' in 'aln' files because I assume that either few people use 'clustalv' anymore, or maybe the file format is the same for it anyway. The change I am suggesting is pretty basic and I will probably edit the source and rebuild for my linux and Mac OS X environments but for this student's Window's machine ( :P) I don't have a compiler and have no interest in getting one.

Thanks (nice program!)

Title: Re: Suggestion for minor change in how UGENE reads Clustal alignments
Post by Mikhail Fursov on Apr 3rd, 2009 at 12:40pm
Hi David!
Thanks for report.

We will include the fix to v1.4.2.

I attached the diff: it changes 2 places in ClustalFormat.cpp: a function with format detection by a file header and format reading function.

Actually I think we need to keep a header in model and reuse it when alignment file is saved. This feature is not in the diff, but it will also be added to the next minor release.
https://forum.ugene.net/forum/YaBB.pl?action=downloadfile;file=clustal.diff (0 KB | )

UGENE Forum » Powered by YaBB 2.5 AE!
YaBB Forum Software © 2000-2010. All Rights Reserved.