Re: [galaxy-dev] picard_MarkDuplicatesWithMateCigar functional test failure

2015-06-12 Thread Nicola Soranzo
  Hi Ryan,
please open a pull request at
https://github.com/galaxyproject/tools-devteam 

The file is in
tools/picard/ directory.

Thanks,
Nicola

Il 12.06.2015 17:23 Ryan G ha
scritto: 

 The functional test for this tool fails. Looking at the XML
wrapper, it looks like $comments is invalidit doesn't exist anywhere
else in the xml file. I know that comments can be specified multiple
times however there is no way to get the comments fields to appear
multiple times in the UI as far as I can tell. So, I changed it from


 #for $element in $comments:
 COMMENT=${element.comment}
 #end
for
 
 to
 
 COMMENT=${comment}
 
 and the functional test
passes.
  


Connetti gratis il mondo con la nuova indoona:  hai la chat, le chiamate, le 
video chiamate e persino le chiamate di gruppo.
E chiami gratis anche i numeri fissi e mobili nel mondo!
Scarica subito l’app Vai su https://www.indoona.com/

___
Please keep all replies on the list by using reply all
in your mail client.  To manage your subscriptions to this
and other Galaxy lists, please use the interface at:
  https://lists.galaxyproject.org/

To search Galaxy mailing lists use the unified search at:
  http://galaxyproject.org/search/mailinglists/

Re: [galaxy-dev] picard_MarkDuplicatesWithMateCigar functional test failure

2015-06-12 Thread Ryan G
done

On Fri, Jun 12, 2015 at 11:33 AM, Nicola Soranzo nsora...@tiscali.it
wrote:

  Hi Ryan,
 please open a pull request at
 https://github.com/galaxyproject/tools-devteam

 The file is in tools/picard/ directory.

 Thanks,
 Nicola

 Il 12.06.2015 17:23 Ryan G ha scritto:

  The functional test for this tool fails.  Looking at the XML wrapper, it
 looks like $comments is invalidit doesn't exist anywhere else in the
 xml file.  I know that comments can be specified multiple times however
 there is no way to get the comments fields to appear multiple times in the
 UI as far as I can tell.  So, I changed it from

 #for $element in $comments:
   COMMENT=${element.comment}
 #end for

 to

 COMMENT=${comment}

 and the functional test passes.








 Connetti gratis il mondo con la nuova indoona: hai la chat, le chiamate,
 le video chiamate e persino le chiamate di gruppo.
 E chiami gratis anche i numeri fissi e mobili nel mondo!
 Scarica subito l’app Vai su https://www.indoona.com/


___
Please keep all replies on the list by using reply all
in your mail client.  To manage your subscriptions to this
and other Galaxy lists, please use the interface at:
  https://lists.galaxyproject.org/

To search Galaxy mailing lists use the unified search at:
  http://galaxyproject.org/search/mailinglists/