On Mon, Aug 15, 2011 at 6:33 PM, Nate Coraor <nate@bx.psu.edu> wrote:
Peter Cock wrote:
On Fri, Jun 3, 2011 at 10:03 AM, Peter Cock <p.j.a.cock@googlemail.com> wrote:
Thanks for the link - I was aware my initial work on adding a generic XML filetype was missing some steps, but I ran out of time yesterday: https://bitbucket.org/galaxy/galaxy-central/issue/568
Hi all,
Could someone please review this patch:
https://bitbucket.org/peterjc/galaxy-central/changeset/83c4366e0641
Hi Peter,
I've merged your branch with these changes in 5897:6165799c4e49. Thanks!
--nate
Thanks Nate :)
This is currently the one and only commit to my xml_filetype branch,
https://bitbucket.org/peterjc/galaxy-central/src/xml_filetype
This attempts to define a new basic data format 'xml' as per issue 568,
https://bitbucket.org/galaxy/galaxy-central/issue/568
This might help with a user error I was just presented with, where an Excel spreadsheet was uploaded and miss-identified as text - when it was clearly XML. Clicking on the 'eye' tried to display it and gave a cryptic error message in the central panel.
In fact, this example makes me wonder if the proposed base XML datatype class should NOT be a subclass of text (as it is now with the blastxml datatype).
Peter
What are your thoughts on the above? I guess for some tasks it makes sense to regard XML as text, but for others not. Peter