[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[GsoC] your latest commit of tasks_udd.py



Hi Akshita,

thanks for your commit.  Unfurtunately my test failed:


$ tasks_udd.py debian-med
task :  bio
+++++++++++++++++++++ 1 bio
----------------- 1 bio
Traceback (most recent call last):
  File "./tasks_udd.py", line 31, in <module>
    tasks.GetAllDependencies()
  File "/home/andreas/debian-maintain/alioth/blends_git/website/webtools/blendstasktools_udd.py", line 840, in GetAllDependencies
    if td.GetTaskDependencies(source):
  File "/home/andreas/debian-maintain/alioth/blends_git/website/webtools/blendstasktools_udd.py", line 1221, in GetTaskDependencies
    query = "EXECUTE query_new('%s')" % List2PgArray(pkgs_not_in_pool)
  File "/home/andreas/debian-maintain/alioth/blends_git/website/webtools/blendstasktools_udd.py", line 486, in List2PgArray
    PgArray=PgArray+komma+'"'+s+'"'
TypeError: cannot concatenate 'str' and 'tuple' objects


after wild guessing the following patch:

 $ git diff
diff --git a/webtools/blendstasktools_udd.py b/webtools/blendstasktools_udd.py
index de18f86..e2fd55c 100644
--- a/webtools/blendstasktools_udd.py
+++ b/webtools/blendstasktools_udd.py
@@ -483,7 +483,7 @@ def List2PgArray(list):
     komma='{'
     PgArray=''
     for s in list:
-        PgArray=PgArray+komma+'"'+s+'"'
+        PgArray=PgArray+komma+'"'+str(s)+'"'
         komma=','
     return PgArray+'}'


It went further until:

$ tasks_udd.py debian-med
task :  bio
+++++++++++++++++++++ 1 bio
----------------- 1 bio
Problem with query
EXECUTE query_new('{"('khmer',)","('rdp-classifier',)","('aegean',)","('kmer',)","('r-cran-phangorn',)","('r-cran-seqinr',)","('express',)","('microbegps',)","('wgsim',)","('sumtrees',)","('ecopcr',)","('barrnap',)","('jaligner',)"}')
FEHLER:  Syntaxfehler bei „khmer“
LINE 1: EXECUTE query_new('{"('khmer',)","('rdp-classifier',)","('ae...
                               ^


Does the script run without any problem at your side?

Kind regards

        Andreas.
-- 
http://fam-tille.de


Reply to: